fix: AGENTIC licence line + a2ml-validate-action repin - #60
Conversation
…, #669) 1. The AGENTIC.a2ml agent-constraint line "Never use AGPL license (...)" contradicts LICENCE-POLICY.adoc Rules 3 (co-developed), 4 (network services) and 5 (games), which MANDATE AGPL-3.0-or-later - and 144 copies named the retired PMPL-1.0-or-later. Replaced with a pointer to the policy plus the A2 no-automated-licence-edits rule, hardcoding no licence so it cannot go stale again. Same wording as the template fix in rsr-template-repo#45; owner-ruled sweep (2026-08-27). 2. Any workflow pinning hyperpolymath/a2ml-validate-action at 59145c7d or e558e79200 is repinned to 6ac6416f. Those two SHAs never existed: the repo itself was only created 2026-08-28 and populated by subtree split from a2ml/actions/validate (286 files, history preserved). The old pins could never resolve and made lockfile generation impossible. Direct push per owner ruling of 2026-08-28. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Up to standards ✅🟢 Issues
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe agent licence constraints now prohibit relicensing and automated licence sweeps. They require agents to read the licence policy, which defines default licences, AGPL requirements, and the PMPL register. ChangesLicence constraints
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested reviewers: Poem
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 |
![codacy-production[bot]](https://avatars.githubusercontent.com/in/56611?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull Request Overview
The pull request successfully updates the licensing instructions to align with the organization's policy regarding AGPL; however, the repinning of a2ml-validate-action mentioned in the title is missing from the file changes. Additionally, there is a path inconsistency in the documentation references, and Rule 2 of the license instructions remains too ambiguous for automated agents to follow effectively. While Codacy reports that quality is up to standards, these implementation gaps and inconsistencies should be addressed before merging.
About this PR
- The changes related to repinning
a2ml-validate-actionare missing from this pull request. Please ensure the migration to the valid SHA is included as intended.
1 comment outside of the diff
[REDACTED:HIGH_ENTROPY]
line 25🟡 MEDIUM RISK
Suggestion: The file path 'LICENCE-POLICY.adoc' is inconsistent with the 'standards/' prefix used on line 26.# sweep (standards/LICENCE-POLICY.adoc A2). New files get correct SPDX from birth.
Test suggestions
- Verify the agent's compliance with Rule 1 (MPL-2.0/CC) for default files.
- Verify the agent correctly identifies scenarios requiring AGPL-3.0 (Rules 3, 4, 5).
- Confirm a2ml-validate-action is repinned to the specified valid SHA.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify the agent's compliance with Rule 1 (MPL-2.0/CC) for default files.
2. Verify the agent correctly identifies scenarios requiring AGPL-3.0 (Rules 3, 4, 5).
3. Confirm a2ml-validate-action is repinned to the specified valid SHA.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
🔍 Hypatia Security ScanFindings: 84 issues detected
View findings[
{
"reason": "Issue in casket-pages.yml",
"type": "missing_timeout_minutes",
"file": "casket-pages.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in casket-pages.yml",
"type": "missing_timeout_minutes",
"file": "casket-pages.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in codeql.yml",
"type": "missing_timeout_minutes",
"file": "codeql.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in dogfood-gate.yml",
"type": "missing_timeout_minutes",
"file": "dogfood-gate.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in dogfood-gate.yml",
"type": "missing_timeout_minutes",
"file": "dogfood-gate.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in dogfood-gate.yml",
"type": "missing_timeout_minutes",
"file": "dogfood-gate.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in dogfood-gate.yml",
"type": "missing_timeout_minutes",
"file": "dogfood-gate.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in dogfood-gate.yml",
"type": "missing_timeout_minutes",
"file": "dogfood-gate.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in hypatia-scan.yml",
"type": "missing_timeout_minutes",
"file": "hypatia-scan.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in instant-sync.yml",
"type": "missing_timeout_minutes",
"file": "instant-sync.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
1 participant
Ruleset refused a direct push, so this lands by PR. Two mechanical fixes, owner-ruled:
Never use AGPL license (…)contradictsLICENCE-POLICY.adocRules 3/4/5 (which mandate AGPL for their scopes). Replaced with the policy pointer used inrsr-template-repo#45. Seestandards#646.a2ml-validate-actionrepin — the previously-pinned SHAs never existed; the repo was only created 2026-08-28 (split froma2ml/actions/validate, history preserved). Repinned to its real HEAD. Seestandards#669.🤖 Generated with Claude Code