arrow_backRetour aux issues
Wpnx330/pharos-cli
#21
Débutant
Ouvrirarrow_forward
Débutant
Ouvrirarrow_forward
Débutant
Ouvrirarrow_forward
bug(doctor): validate TOML and YAML client configs, not just JSON
ecoDébutant
bug
good first issue
descriptionDescription
## Problem
`pharos doctor` validates client config files for syntax errors, but only handles JSON formats. Now that Pharos supports TOML (Codex CLI, Grok Build) and YAML (Aider) configs, `doctor` will either skip them or crash when trying to parse them as JSON.
## Steps to reproduce
1. Install a server to Codex CLI (writes TOML to `~/.codex/config.toml`)
2. Introduce a syntax error in the TOML
3. Run `pharos doctor`
4. Doctor doesn't report the malformed TOML
## Proposed fix
Update `validateConfig()` in `cmd/doctor.go` to dispatch on `client.Format`:
- `FormatMcpServers`, `FormatArray`, `FormatOpenCode`, `FormatZed` → JSON validation
- `FormatTOML` → TOML validation (parse with BurntSushi/toml)
- `FormatHermes`, `FormatAider` → YAML validation (parse with gopkg.in/yaml.v3)
## Acceptance criteria
- [ ] Doctor reports malformed TOML configs (Codex, Grok)
- [ ] Doctor reports malformed YAML configs (Aider, Hermes)
- [ ] Doctor reports malformed JSON configs (all JSON clients) — existing behavior preserved
- [ ] `--json` output includes format type per config check
Issues similaires
medik8s/fence-agents-remediation
star18
Poids du dépôt moyen
v0.8.0 starts a cluster-scoped Secret informer despite namespace-scoped Secret RBAC
## Description While testing Fence Agents Remediation (FAR) v0.8.0, I observed the manager repeatedly attempting to li…
Go
good first issue
Wayfare-labs/wayfare
star1
Poids du dépôt léger
Check that an issuer's home_domain round-trips to the same stellar.toml
Written to the standard set by #33 — read that issue first for the expected level of detail and the acceptance criteria…
Go
help wanted
area:corridor
Wayfare-labs/wayfare
star1
Poids du dépôt léger
Report auth_immutable: whether an issuer's flags can still change
> **This issue is the reference standard.** Every other check issue links back here for the expected level of detail. I…
Go
help wanted
good first issue