Conversation
Update README with v3 architecture and onramp docs. Add biome and knip to CI workflow. Update CONTRIBUTING guide. Add comply-explain to TODOS. Bump package.json dependencies. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Greptile SummaryThis PR updates documentation (README, CONTRIBUTING), CI workflow,
Confidence Score: 4/5Safe to merge after addressing the README/CONTRIBUTING Codex install instructions that reference unimplemented --host flags One P1 finding: README documents ./setup --host codex and CONTRIBUTING documents gen:skill-docs -- --host codex, but neither script implements the --host flag. Users following these instructions will get a silent incomplete install. The fix is straightforward — either implement the flag or add an in-progress callout before merging. README.md (Codex install section) and CONTRIBUTING.md (gen:skill-docs --host codex reference) need attention before merge Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant setup as ./setup script
participant gen as gen-skill-docs.ts
participant agents as .agents/skills/
participant codex as ~/.codex/skills/
Note over User,codex: README-documented Codex flow (currently broken)
User->>setup: ./setup --host codex
Note over setup: --host flag silently ignored
setup->>gen: bun run gen:skill-docs (no --host)
gen-->>setup: SKILL.md files generated (Claude only)
setup-->>User: em-dash ready. exit 0
Note over agents,codex: .agents/skills/ wrappers NOT created
Note over agents,codex: ~/.codex/skills/ NOT linked
Note over User,codex: Claude Code flow (works correctly)
User->>setup: ./setup
setup->>gen: bun run gen:skill-docs
gen-->>setup: SKILL.md files generated
setup-->>User: Skills linked in ~/.claude/skills/
Reviews (4): Last reviewed commit: "fix: remove nonexistent codex test files..." | Re-trigger Greptile |
Remove codex-generation, codex-setup, and codex-smoke test references from package.json and CONTRIBUTING.md since those files don't exist yet. Add biome check and knip steps to CI workflow to match PR description. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Summary
Test plan
🤖 Generated with Claude Code