Repository navigation
Conversation
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Its title has been prefixed with |
This branch has not been deployed
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
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
Commit
30cbc2a4febcffb8c4fec9cd5c9ec25218a4c4ffintentionally removed seven repository instruction files. Cross-platform CI run 38563842325 then failed on five dangling structure-document links and a startup-prompt documentation test that still read the deleted root file. Parent commit732fba65b4cb2ff35c3416a4102553cb36ec28bbpassed Cross-platform CI.This candidate preserves that removal and completes the stale-reference cleanup: the structure-index generator no longer emits the retired rules link or exempts the absent rules file, its fixture no longer creates one, and current documentation/tool messages/comments stop directing contributors to deleted instructions. Existing contributor guidance remains in CONTRIBUTING.md; no retired SOP is copied into a replacement policy file.
The consent-documentation test now verifies the shipped installation guide is included in the package and discoverable from README, instead of requiring development instructions. The guide retains explicit consent, refusal-code, and silence-is-deferred assertions. CLI deferral and management API refusal behavior remain covered by executable tests. Application code changes are comments only. Product support for end users' AGENTS.md files, prompt fixtures, public CLI agent documentation and historical investigation records are retained deliberately.
Validation
Windows, isolated worktree, Bun 1.4.3 (the pinned test runner); locked dependencies installed with lifecycle scripts disabled.
ENOENTlocally before the fix.bun test --isolatecoverage: startup-prompt, star-deferral, sidebar-routes, structure-ssot, ci-structure-gate, setup-hooks, test-layout-tooling, and file-size-ratchet. Final results across those files: 138 passed, 2 existing Windows skips, 0 remaining failures. The first size scan timed out at its unchanged 5-second limit during concurrent work; its isolated rerun passed, with a 641 ms repository scan. The final consent-documentation edit was rerun separately: 14 passed.bun node_modules/typescript/bin/tsc --noEmit: passed.bun scripts/structure-ssot.ts: passed; regenerated INDEX.md is current.bun scripts/generate-ocx-skill-surface.ts --check: passed (all 9 generated files current).bun scripts/privacy-scan.tsandgit diff --check: passed.Full local platform suites were not run for this documentation/generator repair; exact-head hosted CI results will be recorded after completion. Initial sandbox-only test attempts stopped in the test runner's temporary-directory/lock setup, before test execution; the actual runs used the repository's home-isolation and write guards.
Scope
This is an explicitly requested draft candidate against this fork's main, where the deletion occurred. The inherited upstream target gate may reject a main-targeted repair; this PR does not relax that gate. Draft PR #1 and its Bun backport branch remain unchanged. The existing GitHub Pages failure is outside this repair. No merge, release, deployment, running-service change, credential change, or external model review was performed or requested.
Keep this PR draft. Local review inspected the complete change and confirms that application consent checks and runtime behavior were preserved.
Checklist
Remote exact-head validation
Head
33b88b7ed3978d5a94d6a34dac2e64d7693a13b4: Cross-platform CI run 38573926417 completed successfully, including the structure gate, all four Linux test shards and desktop packaged-shell checks. macOS/Windows matrices were skipped by the existing PR selection policy. PR labeler, hygiene, React Doctor and async-contract catalog checks passed. The separate target-branch gate failed withwrong base (main); this candidate intentionally repairs the requested main regression and does not change that gate. No merge or deployment was performed.