Skip to content

feat(agent): add Codex parser diagnostics to agent list - #151

Open
codeaholicguy wants to merge 1 commit into
mainfrom
codex-agent-list-debug
Open

feat(agent): add Codex parser diagnostics to agent list#151
codeaholicguy wants to merge 1 commit into
mainfrom
codex-agent-list-debug

Conversation

@codeaholicguy

@codeaholicguy codeaholicguy commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Simplify agent list parser checks to focus only on true parse failures.
  • Add a small adapter-neutral ParserHealthProvider / SessionParserHealth contract.
  • Implement Codex parser health from the existing normalized session parser.
  • Keep normal agent list output stable; warn only when a listed session has records but yields zero parsed messages.
  • Keep agent list --debug on the existing debug logger pattern and emit only compact failure details.

Validation

  • npm test --workspace @ai-devkit/agent-manager -- CodexAdapter.test.ts: passed, 68 tests.
  • npm test --workspace ai-devkit -- agent.test.ts: passed, 76 tests.
  • npm run typecheck --workspace @ai-devkit/agent-manager: passed.
  • npm run lint --workspace @ai-devkit/agent-manager: passed.
  • npm run lint --workspace ai-devkit: passed with existing unrelated warnings.
  • npm run build --workspace @ai-devkit/agent-manager: passed.
  • npx ai-devkit@latest lint: passed.
  • npm run dev -- agent list --debug from packages/cli: inspected; emitted compact parser failures=0 debug output.
  • Commit hook also ran repo-wide lint/test successfully.

Notes

  • Unknown Codex record/payload types are no longer part of agent list output. The signal is intentionally narrower: session data exists but messages cannot be recovered.

@codeaholicguy
codeaholicguy force-pushed the codex-agent-list-debug branch from a2bbe94 to 4c725e8 Compare August 13, 2026 04:52
@codeaholicguy
codeaholicguy force-pushed the codex-agent-list-debug branch from 4c725e8 to f49fde7 Compare August 13, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant