fix(licence): clear PMPL residue from the files that stay in canon - #648
Merged
Conversation
standards is not a Rule 2 repo -- LICENCE-POLICY.adoc reserves PMPL to exactly three repositories and says every other sole-owner repo, 'INCLUDING THIS standards REPO', is MPL-2.0 / CC-BY-SA-4.0. SCOPE. 471 files carried PMPL text. 390 of them (82%) sit inside k9-svc/, a2ml/, axel-protocol/, rhodium-standard-repositories/ and the other directories that #490-#495 EVICT from this repo. Cleaning those would be work that gets deleted, and would add 390 files of diff noise across carve-out PRs that already move ~1,900 files. They are left for the eviction to carry away. This changes only the 81 that genuinely stay. 52 rewritten, 4 protected, 25 deliberately left (below). DELIBERATELY NOT TOUCHED: * LICENSES/PMPL-1.0-or-later.txt -- the licence exhibit itself * docs/migrations/pmpl-to-mpl-sweep-runbook.adoc -- documents this work * PALIMPSEST.adoc -- documents the header format * scripts/tests/spdx-header-block-test.sh, check-licence-consistency.sh -- tests/checkers that ASSERT on these strings; rewriting them breaks the check rather than fixing anything * LICENCE-POLICY.adoc -- the policy that defines PMPL * docs/audits/* -- HISTORICAL RECORDS. An audit dated 2026-05-26 should keep saying what was true then; editing it falsifies the record Also corrects the agent instruction that read # - Never use AGPL license (use PMPL-1.0-or-later) which after Rule 5 (games are AGPL, added 2026-08-26) contradicts three rules at once -- Rule 3 (co-developed with family), Rule 4 (007) and Rule 5. Replaced with wording that preserves the MPL default without telling agents to undo deliberate policy. The estate-wide instance of this, in ~899 generated AGENTIC.a2ml files, is tracked separately in #646 because it needs a source fix rather than a sweep. Note the SPDX headers on these files were already MPL-2.0 throughout, so the legally operative licence has been correct the whole time; this is metadata hygiene, not a licence change. Every touched JSON/YAML/TOML/sh file re-verified to parse. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Warning Review limit reachedNext included review available in 52 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (77)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
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.
standardsis not a Rule 2 repo.LICENCE-POLICY.adocreserves PMPL to exactly three repositories and states every other sole-owner repo — "including thisstandardsrepo" — is MPL-2.0 / CC-BY-SA-4.0.Scope: 82% of it is leaving anyway
471 files carried PMPL text. 390 of them sit inside
k9-svc/,a2ml/,axel-protocol/,rhodium-standard-repositories/and the other directories that #490–#495 evict from this repo.Cleaning those would be work that gets deleted — and would add 390 files of diff noise across carve-out PRs that already move ~1,900 files. Left for the eviction to carry away.
This changes only the 81 that genuinely stay: 52 rewritten, 4 protected, 25 deliberately left.
Deliberately not touched
LICENSES/PMPL-1.0-or-later.txtdocs/migrations/pmpl-to-mpl-sweep-runbook.adocPALIMPSEST.adocspdx-header-block-test.sh,check-licence-consistency.shLICENCE-POLICY.adocdocs/audits/*Also fixes an agent instruction that contradicts three rules
After Rule 5 (games are AGPL, added today) this contradicts Rule 3 (co-developed with family), Rule 4 (
007) and Rule 5. Replaced with wording that keeps MPL as the default without instructing agents to undo deliberate policy.The estate-wide instance — ~899 generated
AGENTIC.a2mlfiles — is tracked separately in #646, because it needs a source fix rather than a sweep.Not a licence change
SPDX headers on these files were already
MPL-2.0throughout, so the legally operative licence has been correct the whole time. This is metadata hygiene. Every touched JSON/YAML/TOML/shell file was re-verified to parse.🤖 Generated with Claude Code