Skip to content

chore: dogfood init — add CI adoption files to the repo#35

Open
bntvllnt wants to merge 1 commit into
mainfrom
chore/dogfood-self-adoption
Open

chore: dogfood init — add CI adoption files to the repo#35
bntvllnt wants to merge 1 commit into
mainfrom
chore/dogfood-self-adoption

Conversation

@bntvllnt
Copy link
Copy Markdown
Owner

Dogfoods the init command shipped in #34, verified against the published canary 2.3.0-canary.8cfac95.

Running codebase-intelligence@canary init . on this repo generated the agent-adoption files. The repo now practices what it ships — AI agents working here are told to query codebase-intelligence before grep/read.

Files

  • AGENTS.md, CLAUDE.md — managed instruction block appended (existing content untouched)
  • GEMINI.md, CONVENTIONS.md, .cursor/rules/codebase-intelligence.mdc, .github/copilot-instructions.md — new

Pure additions (151 insertions, 0 deletions). Docs/agent files only — no src/** change, so this does not trigger a canary publish on merge.

Verification

init from the published canary: 6 files written, pre-existing content preserved, re-run fully idempotent, skill install confirmed (temp HOME). End-to-end pipeline (merge → OIDC canary publish → npx @canary → init) works.

Generated by `codebase-intelligence@canary init` (the feature from #34).
The repo now instructs its own AI agents to query CI before grep/read.

- AGENTS.md, CLAUDE.md: managed instruction block appended
- GEMINI.md, CONVENTIONS.md, .cursor/rules/, .github/copilot-instructions.md: new
- docs/agent files only — no src change, does not trigger a canary publish
Copy link
Copy Markdown
Owner Author

@bntvllnt bntvllnt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review — 0 findings (0 blocking, 0 warn)

BLOCKING

None.

WARN

None.

VERIFIED CLEAN

  • Reviewed all changed files at head e352f773707d74246b11cd970a1e80a8917b69c1: .cursor/rules/codebase-intelligence.mdc, .github/copilot-instructions.md, AGENTS.md, CLAUDE.md, CONVENTIONS.md, GEMINI.md.
  • The PR is bounded to repo/agent instruction adoption files only; no src/**, package metadata, release, or runtime behavior changes are included.
  • The inserted managed blocks match the single source of truth in src/install/index.ts#renderBlock, preserve existing AGENTS.md / CLAUDE.md content, and use valid CLI commands present in src/cli.ts.
  • The additions align with the VLLNT Platform dogfood objective: this repo now consumes the init adoption layer it ships, without adding internal/private operational details.
  • Manual approval recommended. Per reviewer policy, I am not submitting an autonomous APPROVE.

VALIDATION

  • Live PR state re-fetched immediately before review: open, non-draft, base main, head e352f773707d74246b11cd970a1e80a8917b69c1, mergeStateStatus=CLEAN, mergeable=MERGEABLE, no prior reviews/comments observed.
  • GitHub checks observed: Quality Gates (18) pass; Quality Gates (22) pass.
  • Ran locally after pnpm install --frozen-lockfile:
    • git diff --check origin/main...HEAD — pass
    • pnpm lint — pass
    • pnpm typecheck — pass
    • pnpm build — pass
    • pnpm test — pass: 17 files, 310 passed, 3 todo
  • Ran built CLI smoke/idempotency check: node dist/cli.js init <temp-repo> --no-skill --json created/updated the six target files on first run, reported all six unchanged on second run, preserved pre-existing AGENTS.md content.

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