Refresh canonical skill-lib snapshot - #74
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3dec3cf490
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ```text | ||
| every admitted character is a gonol | ||
| characters -> words -> definitions -> recursive gonol relations | ||
| ``` | ||
|
|
||
| This is an **EDCM text-domain rule**, not a UCNS linguistic rule and not a | ||
| METAPAT axiom. | ||
| This order is load-bearing. |
There was a problem hiding this comment.
When an agent follows this required skill for suffix or suffix-coupling work, this load-bearing four-stage ladder contradicts EDCM's implemented declared-scale-option-set model: suffix gonols can close directly, and closed gonols may participate at any admissible scale. This guidance can therefore reject already-supported edcm.gonol construction paths; retain the option-set model instead of making this sequence exclusive.
AGENTS.md reference: AGENTS.md:L107-L107
Useful? React with 👍 / 👎.
| `The-Interdependency/skill-lib`. | ||
|
|
||
| Source commit: `a1c6a7124af537ee9937b6fc6084940091982fe5` | ||
| Source commit: `c14ee9d500579a4b5d6821f62c9d82ca96e73608` |
There was a problem hiding this comment.
In the inspected .github/workflows/skill-compliance.yml, both the canonical checkout and strict drift checker remain pinned to a1c6a7124af537ee9937b6fc6084940091982fe5; changing the vendored bytes and this declaration to c14ee9d... therefore guarantees the byte-for-byte drift job fails. Update the workflow and repository authority references to the new identity, or retain the old snapshot.
AGENTS.md reference: AGENTS.md:L35-L41
Useful? React with 👍 / 👎.
| ], | ||
| "gaps": [], | ||
| "repo": "The-Interdependency/edcm" | ||
| "repo": "edcm" |
There was a problem hiding this comment.
The inspected msdmd workflow invokes the collector with --repo The-Interdependency/edcm and then diffs its output byte-for-byte against this tracked file. Changing the tracked value to edcm guarantees that comparison fails with this one-line difference, independently of the skill-lib pin; regenerate using the workflow command or update the command and identity contract consistently.
Useful? React with 👍 / 👎.
| A completion claim requires: | ||
|
|
||
| ## Anti-patterns: refuse these | ||
|
|
||
| - Assigning character, morphology, definition, or lexical-source authority to | ||
| UCNS merely because UCNS provides their geometry. | ||
| - Assigning affixiation semantics to EDCM or UCNS merely because they implement | ||
| or apply it. | ||
| - Reopening closed gonols merely because a larger construction contains them. | ||
| - Inferring function context, syntax, precedence, or operation from adjacency, | ||
| Unicode names, dictionary glosses, or glyph shape. | ||
| - Treating a source word list as a semantic dictionary. | ||
| - Treating `hmmm` or an unresolved constructor as permission to refuse a | ||
| declared candidate construction, complete run, or replay. | ||
| - Inventing morphology, a lexical family map, geometry, measurement authority, | ||
| alternate carrier, scale rule, or coupling law to fill an absent constructor. | ||
| - Promoting a sidecar relationship graph into intrinsic gonol semantics. | ||
| - Treating a partial corpus run as complete evidence. | ||
| - Adding arbitrary wall-clock limits to a healthy admitted computation. | ||
| - Restoring the deprecated omega/phi/psi cores, bone/flesh categories, fixed | ||
| morphology weights, or `multiplyFuel`/carrier-LCM as the universal language | ||
| law. | ||
| - Transferring UCNS construction survival into EDCM measurement validity, | ||
| PTCNA efficacy, cognition claims, or selected universal canon. | ||
|
|
||
| ## Minimal examples | ||
|
|
||
| Character admission: | ||
| 1. exact UCNS, EDCM, source/profile, and constructor identities; | ||
| 2. the complete declared source scope; | ||
| 3. deterministic construction receipts; and | ||
| 4. independent complete replay where replay is required by the governing protocol. |
There was a problem hiding this comment.
For any completed construction that applies affixiation, including suffix coupling, this checklist now permits a receipt containing only UCNS and EDCM identities even though METAPAT is the governing semantic authority. Replays against different METAPAT contracts could therefore satisfy the listed completion requirements while losing required provenance; include the exact governing METAPAT identity whenever affixiation is used.
AGENTS.md reference: AGENTS.md:L35-L41
Useful? React with 👍 / 👎.
| ".rb": "#", ".rake": "#", ".gemspec": "#", | ||
| ".ex": "#", ".exs": "#", | ||
| ".sh": "#", ".bash": "#", ".zsh": "#", ".fish": "#", | ||
| ".pl": "#", ".pm": "#", ".t": "#", |
There was a problem hiding this comment.
.pl extension
The new registry assigns every .pl file to Perl's # marker, but .pl is also a conventional Prolog extension and Prolog metadata uses %, which this registry recognizes only for .prolog. Consequently, parse_file silently misses valid % === ... blocks in ordinary Prolog .pl sources and walk_tree reports them as unannotated, contrary to the newly documented rule that ambiguous suffixes are refused rather than guessed; remove .pl auto-detection or require an explicit language choice, and mirror the fix in the TypeScript registry.
Useful? React with 👍 / 👎.
1 participant
Propagation
The-Interdependency/skill-libc14ee9d500579a4b5d6821f62c9d82ca96e73608The-Interdependency/edcmeb5f200d48a8c4ffa7b943238407fbdac4934946canon,domain-claims,gonol-build,interdependent-work-graph,meta-module-build,msdmd,test-build,the-interdependencyVerification
Generated files
edcm_msdmd.tsDoctrine / hmmm
hmmm, not a claimed passFollow-up audit
.github/workflows/skill-compliance.ymlfrom the prior canonical pin toc14ee9d500579a4b5d6821f62c9d82ca96e73608.edcm_msdmd.tswith repository identityThe-Interdependency/edcm.