fix(a2ml): dedupe — keep descriptiles/, retire the 6a2/ copies - #28
fix(a2ml): dedupe — keep descriptiles/, retire the 6a2/ copies#28hyperpolymath wants to merge 1 commit into
Conversation
Owner ruling 2026-08-27: 6a2/ is retired; descriptiles/ is canonical. descriptiles holds the files that DESCRIBE the repo's state; contractiles holds the contracts being honoured. Where both directories held a file, the descriptiles copy is kept. ⚠ EVERY REMOVAL IS ACCOUNTED FOR BELOW. The copies differed - they were not duplicates - so this is not a no-op cleanup: AGENTIC (both real; removed 6a2 34L, kept descriptiles 51L) ECOSYSTEM (removed 6a2 copy was REAL, 22L; kept descriptiles copy is TEMPLATE RESIDUE, 45L) META (both real; removed 6a2 27L, kept descriptiles 53L) NEUROSYM (both real; removed 6a2 21L, kept descriptiles 23L) PLAYBOOK (both real; removed 6a2 26L, kept descriptiles 137L) STATE (removed 6a2 copy was REAL, 38L; kept descriptiles copy is TEMPLATE RESIDUE, 64L) Two of these are a real loss: the removed 6a2 copy held this project's own content while the kept descriptiles copy is un-instantiated rsr-template-repo residue. That is what the ruling selects for, and it is recorded here rather than dropped silently. The removed content remains in git history at the parent commit. Estate-wide this affects 5 repos: 007, 007-lang, frayed-knot-toolkit, lithoglyph, _pathroot. Measured across all of them: prefer-descriptiles loses real content in 19 file-pairs, drops a second real copy in 29 more, and correctly drops template residue in ZERO cases.
|
Warning Review limit reachedNext included review available in 56 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 (6)
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 |
Up to standards ✅🟢 Issues
|
There was a problem hiding this comment.
Pull Request Overview
This PR executes the retirement of the .machine_readable/6a2/ directory in favor of .machine_readable/descriptiles/ as the canonical source for repository state. Codacy analysis indicates the changes are up to standards, as the PR primarily consists of file deletions.
However, a significant risk exists regarding broken references. The current diff only removes files without updating automation, CI/CD configurations, or code that may still point to the 6a2/ path. Additionally, this consolidation results in a regression of metadata quality for ECOSYSTEM and STATE files, as project-specific content is replaced by 'template residue' to maintain structural naming compliance. These items should be verified before merging to ensure no disruption to A2ML-dependent workflows.
About this PR
- The PR contains file deletions but no updates to code or configuration files that might reference the '.machine_readable/6a2/' path. This could break automation or CI/CD tools that consume these A2ML files.
- Note that this change accepts a regression in metadata quality for 'ECOSYSTEM' and 'STATE' files, replacing project-specific content with 'template residue' to satisfy directory naming standards.
Test suggestions
- Verify that all automation or CI/CD tools consuming A2ML files are updated to look in 'descriptiles/' instead of '6a2/'.
- Verify that the 'descriptiles/' directory in this repository actually contains the intended canonical files.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that all automation or CI/CD tools consuming A2ML files are updated to look in 'descriptiles/' instead of '6a2/'.
2. Verify that the 'descriptiles/' directory in this repository actually contains the intended canonical files.
Low confidence findings
- While the description mentions affecting 5 repositories, this PR only covers deletions within this specific context. Ensure the wider estate 'real loss' has been audited.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Owner ruling 2026-08-27:
6a2/is retired;descriptiles/is canonical — it holds the files that describe the repo's state, whilecontractiles/holds the contracts being honoured.Where both directories held a file, the
descriptilescopy is kept.⚠ Every removal is accounted for
The copies differed — they were not duplicates — so this is not a no-op cleanup:
Two of these are a real loss: the removed
6a2copy held this project's own content, while the keptdescriptilescopy is un-instantiatedrsr-template-reporesidue. That is what the ruling selects for, and it is recorded here rather than dropped silently. The removed content remains in git history at the parent commit.Estate context
This affects 5 repos:
007,007-lang,frayed-knot-toolkit,lithoglyph,_pathroot. Measured across all of them, prefer-descriptiles loses real content in 19 file-pairs, drops a second real copy in 29 more, and correctly drops template residue in zero cases.Related: standards#655, phronesis#92, proof-burrower#75.