feat(docs+fm-10): v1 architecture docs, pre-commit auto-install, ADR-0086/0087#352
Open
ElCuboNegro wants to merge 6 commits into
Open
feat(docs+fm-10): v1 architecture docs, pre-commit auto-install, ADR-0086/0087#352ElCuboNegro wants to merge 6 commits into
ElCuboNegro wants to merge 6 commits into
Conversation
…DR-0085) Changes: - tools/adr_gate.py: _has_pending_adr() now checks `git log adrs -- docs/adr/` instead of working-tree status, preventing agents from staging ADR+code in the same commit. Falls back to working-tree check when adrs branch does not yet exist. Adds 'adrs' to _EXEMPT_BRANCH_PREFIXES. - tools/enforce_gitflow.py + cornerstone/runtime/gitflow.py: recognise 'adrs' as a valid source for staging PRs (ADR-0050 parity). - All 5 starter hooks (post_gen_project.py): add _create_adrs_branch() that creates the adrs ledger branch after the first git commit in generated repos. - AGENTS.md: document the ADR-first branch workflow as §6 (permanent adrs ledger, step-by-step mandate, gate behaviour). Renumber §6→§7 and §7→§8. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tor (ADR-0086) - cornerstone/runtime/init_cmd.py: _install_precommit() runs on every cornerstone init — covers the cloner case (clone + cornerstone init activates all quality gates automatically). - cornerstone/runtime/doctor.py: _check_precommit_hooks() warns when .git/hooks/pre-commit is absent; --repair installs them automatically. - Starters base/cli/mcp/api post_gen_project.py: _install_precommit() called after _create_adrs_branch() — covers new project generation. Also fixes api starter which had _create_adrs_branch defined but not called (missed in ADR-0085 commit). pipeline starter already had this pattern and is unchanged. All calls are best-effort: FileNotFoundError is silently skipped so projects without pre-commit in PATH are not blocked. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…-0085) 18 new files (8,740 lines): 7 per-repo ecosystem specs (cornerstone-agents, crisol-gateway, crisol-mcp-github, crisol-context-mcp, crisol-bigquery-mcp, keystone, cornerstone-server), 5 L0–L4 guides (getting-started, cli-reference, starters-and-skills, pipelines, governance-and-telemetry), 5 implementation specs for open failure modes (FM-01/02/03/05/06), and PRE_MORTEM.md. mkdocs.yml wired with Guides and Implementation Specs nav sections. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Records the full v1 ecosystem topology: 7-repo layout, 5-layer architecture, 8 starters, 5 pipelines, ADR-first branch workflow (adrs ledger branch), community-edition constraints, provider neutrality, and semver policy. This is the first commit on the permanent `adrs` ledger branch — the ADR was written before any feature branch for the enforcement changes is created, per the workflow mandated in §5 of this ADR itself. [skip-adr] Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…n protocol community ADR-0086 (FM-10): mandates pre-commit install in cornerstone init and all starter hooks — eliminates the manual step that leaves cloners without local quality gates. ADR-0087 (FM-14): supersedes ADR-0063 with a deployer-agnostic MCP catalog registration protocol. Separates the community protocol definition from the crisol-gateway OSS implementation. Also embeds the FM-07 fix as a protocol requirement: registration CI MUST NOT use continue-on-error. [skip-adr] Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ElCuboNegro
force-pushed
the
feature/ADR-0086-precommit-auto-install
branch
from
639a788 to
1b1e6a0
Compare
ElCuboNegro
force-pushed
the
feature/ADR-0086-precommit-auto-install
branch
from
1b1e6a0 to
25fa98a
Compare
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
1 participant
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
This branch bundles three cohesive deliverables for the Community v1 beta:
FM-10 — Pre-commit auto-install (ADR-0086)
cornerstone initnow callspre-commit install --install-hooks+--hook-type commit-msgafter project setupcornerstone doctorgains_check_precommit_hooks(): warns if hooks absent, repairs with--repair_install_precommit()— matches the_create_adrs_branch()patternFM-14 — MCP Registration Protocol decoupling (ADR-0087)
crisol-registerCI job MUST NOT usecontinue-on-error: truev1 Architecture Documentation (ADR-0085)
docs/specs/,docs/guides/,docs/implementation/mkdocs.ymlwired withGuides:andImplementation Specs:nav sectionsadrsbranch so mkdocs builds cleanlyTest plan
mkdocs build --strictexits 0, no broken anchorscornerstone initin a fresh git repo →pre-commithooks installedcornerstone doctorin a repo with no hooks → reports warning;--repairfixes itCloses #334 (FM-10), #341 (FM-14)
🤖 Generated with Claude Code