chore(claude): share Claude Code and agent configuration#147
Merged
Conversation
Make the local Claude Code / agent workflow configuration a shared, committed part of the repo so the whole team gets the same skills, project guidance, and review hooks. Changes: - Add .claude/skills and mirrored .agents/skills (13 subsystem skills: fhirpath, sof, pysof, hts, run-hfs-server, test-hfs, bulk-data export/submit, docker-and-release, plus new auth, audit, subscriptions, and cds-hooks). - Add .claude/settings.json and .claude/hooks: the hfs-policy stop-gate plus the Codex-backed plan gate (PreToolUse/ExitPlanMode), final implementation review (Stop), and a shared logging helper, with .claude/schemas holding their JSON output contracts. - Refresh CLAUDE.md: correct the crate count to 17 and add the hts/auth/audit/subscriptions/cds-hooks crate rows and skill entries. - Add AGENTS.md and stop ignoring .claude / AGENTS.md in .gitignore; keep the .claude/state and .claude/debug runtime dirs ignored. The Codex hooks are fail-closed and require the `codex` CLI; set CODEX_PLAN_GATE_DISABLE=1 to bypass them. Tests: docs/config only. Validated shell syntax (bash -n) and JSON (jq) for all hooks and schemas locally; no compiled code changed.
smunini
previously approved these changes
Jun 13, 2026
…ents-config # Conflicts: # CLAUDE.md
smunini
approved these changes
Jun 13, 2026
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Promotes the local Claude Code / agent workflow configuration to a shared, committed part of the repository so every contributor gets the same project guidance, subsystem skills, and review hooks. Previously
.claude/andAGENTS.mdwere gitignored and lived only on individual machines.Changes
Skills (
.claude/skills/+ mirrored.agents/skills/)run-hfs-server,work-with-fhirpath,work-with-sof,work-with-pysof,work-with-hts,test-hfs,bulk-data-export,bulk-data-submit,docker-and-release, plus newwork-with-auth,work-with-audit,work-with-subscriptions, andwork-with-cds-hooks.Project guidance
CLAUDE.md: corrected the workspace crate count (→ 17) and added the missinghelios-hts,helios-auth,helios-audit,helios-subscriptions, andhelios-cds-hookscrate-table rows and skill entries.AGENTS.md: added.Review hooks (
.claude/settings.json,.claude/hooks/,.claude/schemas/)hfs-policy.sh— session/prompt context plus a Stop gate that holds completion untilcargo fmt --all, CI-style clippy, and an affectedcargo testare recorded.codex-verify-plan.sh— PreToolUse gate onExitPlanMode; Codex reviews the plan against an HFS-specialized schema (domains: server_api, persistence, fhir_core, subscriptions, tooling_config) covering tenant-first persistence, FHIR-version gating, CI coverage, and reuse-over-duplication.codex-final-plan-review.sh— Stop-time review comparing the accepted plan to the implementation (git diff + recorded check state), capped retries, with remote-CI execution treated as out of scope.codex-gate-log.sh— shared timestamped + JSONL logging used by both gates..gitignore
.claude/andAGENTS.md; keep.claude/state/and.claude/debug/(runtime) ignored.Testing
[skip ci]).bash -non all hook scripts andjq emptyon all schemas pass; the gate pass/deny/block paths were exercised locally with stubbed and realcodexruns.Notes
codexCLI. Contributors without it will be blocked atExitPlanMode(plan gate) andStop(final review) unless they setCODEX_PLAN_GATE_DISABLE=1.hfs-policy.shonly needsjq/python3..claude/skills/and.agents/skills/are kept as identical mirrors.