Repository navigation
governance: constitution, enforcement table and ADRs 0001-0003 (draft for review) - #578
Draft
mlieberman85 wants to merge 1 commit into
Draft
mlieberman85 wants to merge 1 commit into
mlieberman85 wants to merge 1 commit into
Conversation
Draft for maintainer review. Adopts the swansong governance model: a short constitution whose rules each carry an enforcement id, an ENFORCEMENT.md recording each check's honest status, where-decisions-live.md, and immutable ADRs. - ADR-0001: darnit aligns a project's setup with a hygiene goal; it does not verify runtime behavior or tool outcomes. Admission test. - ADR-0002: governance model; Spec Kit constitution becomes a pointer; framework-design.md stops being authoritative; governance/ is not a Spec Kit feature. - ADR-0003: docs consolidation plan (contracts, design notes, user docs, archive, deletions) and the contradictions to decide first. Nothing outside governance/ changes in this commit. Assisted-by: Claude:claude-opus-5-5 Signed-off-by: Michael Lieberman <mlieberman85@gmail.com>
Marc-cn
reviewed
Oct 10, 2026
Marc-cn
left a comment
Collaborator
There was a problem hiding this comment.
Agree with the direction and with R1 first. On the five decisions:
- Scope. zizmor controls: reframe to reading the workflow file. The zizmor result can change with the zizmor version, and with no workflows it is an exit-code WARN (#584). Threat-model engine and Scorecard wiring, don't havea strong view.
- Model judgments. Keep them, as PASS candidates a person confirms. That is how it behaved when I ran an audit through LLMs.
- "Set up for". Yes.
- Contradictions. Unknown step keys fail loading: yes (
--profilewas accepted and ignored, #576). Remediation on FAIL only: yes. - Rule count. No opinions. Two bugs this week fit none of the 18: undeclared dependencies (#573) and the CLI and MCP paths giving different answers (#576, #577).
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
Draft for maintainer review. This PR proposes a governance model for darnit, adapted from swansong's
governance/:ENFORCEMENT.md, which records each check's honest status;where-decisions-live.md, which routes each kind of decision to its document;Nothing outside
governance/changes. CLAUDE.md, the Spec Kit constitution andframework-design.mdare untouched until the ADRs are accepted.What to review:
governance/decisions/0001-what-darnit-is.mddecides what darnit is for. darnit is a framework that aligns a project's setup with a hygiene goal, and it does not verify runtime behavior or tool outcomes. For example, it checks that gittuf is set up, not that history passes gittuf policy. It also defines a three-question admission test and lists the consequences, including which controls are cut or reframed.CHARTER.mdis the LF Technical Charter.governance/constitution.mdholds 18 rules:governance/ENFORCEMENT.mdgives one row per rule, with an honest status and a build order.maincurrently requires no passing checks and no approvals, so CI gates nothing; R1 is first in the build order.governance/decisions/0002-governance-follows-swansong.mdcovers four changes:framework-design.mdstops being "authoritative";governance/is not a Spec Kit feature (as swansong's ADR-0006 decided).governance/decisions/0003-docs-consolidation.mdgives a disposition for every doc, a section-by-section split offramework-design.md, and the contradictions to resolve first.Decisions needed
[plugins]table: moves out of framework TOML;.baseline.tomlpromise: superseded by feat!: stop reading a repository's .baseline.toml #556.Type of Change
Framework Changes Checklist
docs/architecture/framework-design.md) if behavior changed. Not applicable: no behavior changes, and ADR-0002 proposes retiring this checkbox.uv run python scripts/validate_sync.py --verboseand it passes. No code changed.Testing
uv run pytest tests/ -v). Not applicable: docs only.uv run ruff check .)AI assistance
Claude (Claude Code, claude-opus-5-5) drafted all of these documents at my request. The inputs were:
This description was also drafted with Claude. The commit carries an
Assisted-by: Claude:claude-opus-5-5trailer.🤖 Generated with Claude Code