Skip to content

refactor(ci): carve the CI action sources out to rac-ci [roadmap:rac-ci]#244

Open
tcballard wants to merge 2 commits into
mainfrom
claude/repo-topology-convergence-2a1awb
Open

refactor(ci): carve the CI action sources out to rac-ci [roadmap:rac-ci]#244
tcballard wants to merge 2 commits into
mainfrom
claude/repo-topology-convergence-2a1awb

Conversation

@tcballard

Copy link
Copy Markdown
Collaborator

PR-2 of 2 for the rac-ci move (companion to the now-merged asdecided/ci#1). Removes the CI action sources from rac-core and rewires the dogfood, per the rac-ci roadmap and ADR-096.

What this does

  • Removes the in-repo wrappers now living in rac-ci: action.yml (watchkeeper), pr-gate-action/ (gatekeeper), validate-action/ (registrar), the reusable .github/workflows/watchkeeper.yml, and the two action-contract tests (those moved to rac-ci).
  • Rewires the dogfood in pr-checks.yml: the watchkeeper / validate / gate jobs now run rac watchkeeper / rac validate --sarif / rac gate --sarif as direct source-install CLI steps (pip install -e .), mirroring the existing agent-rules/eval/doctor jobs. rac-core dogfoods the engine itself, not its own published wrappers — no dependency on a rac-ci tag.
  • Drops the two action tests from the watchkeeper battery in tests.yml (the CLI tests test_watchkeeper.py / test_gate.py stay).
  • Docs (watchkeeper.md, validation.md) point consumers at the rac-ci paths (itsthelore/rac-ci/<cap>/github@v1).
  • Adds ADR-096 (rac-ci consolidation: capability × platform, source-install dogfood, v1 tagging) and links the rac-ci roadmap to it.

Versioned cutover

Old rac-core action references stay resolvable on their existing tags (itsthelore/rac-core@<oldtag>, …/validate-action@v0, …/pr-gate-action@v0); only @main and future tags drop the actions.

Gates (all green locally)

  • rac validate rac/ exit 0 (332 artifacts)
  • rac relationships rac/ --validate exit 0
  • rac export rac/ --agent-rules --check exit 0
  • rac review rac/ no priority 1–2 (health 94/100)
  • Both workflows YAML-parse; the rewritten dogfood jobs run on this very PR.

Note

This PR's own pr-checks.yml run is the live test of the rewritten source-install dogfood jobs. main is protected, so it needs your review/approval. Merge with a merge commit.

Execution tracking: #228 (rac-ci item — completes it alongside rac-ci#1).


Generated by Claude Code

Implements rac/roadmaps/repo-topology/rac-ci.md (ADR-092, ADR-096).

The watchkeeper/gatekeeper/validate composite actions and the reusable
watchkeeper workflow now live in itsthelore/rac-ci (capability x platform).
Removes the in-repo sources (action.yml, pr-gate-action/, validate-action/,
the reusable workflow, and the two action-contract tests) and repoints the
dogfood: pr-checks.yml now runs rac watchkeeper / rac validate --sarif /
rac gate --sarif as direct source-install CLI steps, mirroring the
agent-rules/eval/doctor jobs, so rac-core dogfoods the engine itself rather
than its own published wrappers. Docs point consumers at the rac-ci paths.
Old rac-core action tags stay resolvable (versioned cutover).
…-ci]

Implements rac/roadmaps/repo-topology/rac-ci.md.

Records the rac-ci decisions: capability x platform layout
(watchkeeper/gatekeeper/registrar + recordkeeper placeholder), wrappers
install the published rac-core (source-install mode dropped), rac-core
dogfoods via direct source-install CLI steps, and rac-ci is versioned with a
moving major v1 tag (ADR-076 CalVer is for PyPI releases; rac-ci ships no
package). Links the rac-ci roadmap to ADR-096 and regenerates the agent-rules
managed blocks.

tcballard commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator Author

Flagging as the remaining blocker for the rac-ci topology item (#238): rac-ci#1 is merged, so this rac-core half is what still holds the carve-out open — epic #228 already shows rac-ci ticked. It would be good to review/merge (protected main, maintainer approval needed) before cutting the next release, so the release ships the rewired source-install dogfood rather than the removed wrapper paths. Note the branch is now ~186 commits behind main and will need a rebase; the in-PR "ADR-096" numbering has since been taken (ADR-096 is external-target verification; rac-ci consolidation prose on main cites ADR-092), so the ADR file needs renumbering on rebase as the PR body anticipated.

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