Repository navigation
Pin the claude-native-practice skill as a first-party manifest row and raise the listing ceiling to 11,000 - #972
Merged
seathatflowsinourveins merged 1 commit intoOct 10, 2026
Conversation
…d raise the listing ceiling to 11,000 The command center's ruling (option (a) of four, 2026-10-10): the repository's own claude-native-practice skill (#923, #958) becomes a row of adoption/skills/manifest.json at #958's landed commit, listed on for Claude Code and enabled in Codex, and the manifest's own ceiling on the on-listed description sum moves from 10,500 to 11,000 characters. The sum goes from 10,374 to 10,723 (+349, 277 characters left), the Codex enabled and catalog sums from 10,055 to 10,404, and the Codex catalog from 27 to 28 skills, 3,171 estimated tokens against the 6,000 setting. The row costs about 90 tokens in every session whose listing includes it. The row is computed from the landed commit's own bytes (tree, SKILL.md sha256 and size, description length) and carries first_party: true, because a skill of this repository has no skills.sh page: its audits.url says it is reviewed in the pull request that changes it. tests.test_skills_manifest holds a first-party row to this repository as its source and to that form while every other row keeps the skills.sh requirement. Follows from the row: the Claude template's skillOverrides entry, the Codex template comment's count, the runtime-worker exclusion (adoption_ref, with a reason and an overturn) and its README count, the landscape catalog's source and its agent-docs task, and the dispositions row that cites a manifest line. The decision record gives the listing check, the three declared test changes, the inverse and the revisit (the ceiling comes back down when the wave's skills-slot verdicts land). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
Dependency limit exceeded — report not shown. This pull request scan exceeded the 10,000-dependency limit applied to this scan, so the results are incomplete and may be inaccurate. To avoid reporting false positives, Socket has not posted a report. Upgrade your plan to raise the dependency limit and get complete reports, or view the partial scan in the dashboard. Socket is always free for open source. If this is a non-commercial open source project, contact us to request a free Team account. |
seathatflowsinourveins
deleted the
claude/cc-native-practice-skills-row-20261010
branch
October 10, 2026 18:53
This was referenced Oct 10, 2026
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.
Scope
claude-native-practiceskill in the central skills manifest as a first-party row (listedonfor Claude Code, enabled in Codex) at Ground the Claude Code native practice in primary sources and one cross-client mapping #958's landed commit, and raises the manifest's own ceiling on the on-listed description sum from 10,500 to 11,000 characters, as the command center ruled (option (a) of four, 2026-10-10). The row costs about 90 tokens in every session whose listing includes it.9d4c00cd05d79b3fd5074df2e46bacab87ee67ff(main, which has Ground the Claude Code native practice in primary sources and one cross-client mapping #958 landed); the row'srefis Ground the Claude Code native practice in primary sources and one cross-client mapping #958's landed commit7a6bfbcf0a4f7201c64e5410c2ccbed1b224b511and itstree_shathe tree of.claude/skills/claude-native-practicethere.lane:foundationadoption/skills/manifest.json(the row, the three description sums, the ceiling intrial.on_description_char_capandbudget.claude_on_cap, the Codex catalog count inclient_budgets),adoption/templates/claude.settings.template.json(skillOverridesentry),adoption/templates/codex.config.template.toml(the comment's count),blueprints/runtime-workers/skills/manifest.jsonandREADME.md(the worker exclusion and the reuse count),catalogs/landscape/skills-lifecycle.json(the source and theagent-docstask),catalogs/foundation/upstream-surface-dispositions.json(one cited manifest line moved),docs/decisions/2026-10-10-skills-claude-native-practice-row.md(new), three tests,manifests/evidence.json.Revisit note (the ruling's): the ceiling comes back down, to 10,500 if the on-listed sum then fits, when the 2026-10-10 wave's thirteen skills-slot verdicts land; I did not read that wave's verdict status for this PR. After this lands the designated host shell installs the skill by
adoption/skills/lifecycle.md; a pin does not install itself.SOTA sources
.claude/skills/claude-native-practiceof this repository at7a6bfbcf0a4f7201c64e5410c2ccbed1b224b511(tree3a485d2a4bbe63dc1ae54e23101c472d21450b8c,SKILL.mdsha256a52a24bc7691fa18b3e4c532c6cbd15b79968d82a54acb4202c9fcced2ad8bcb, 2,224 bytes, description 349 characters), added by Claude Code native practice, 2026-10: one evidenced default per role slot #923 and extended by Ground the Claude Code native practice in primary sources and one cross-client mapping #958; no body is copied.skillListingBudgetFraction(default 0.01, the template sets 0.05 under the 2026-09-30 directive), the 8,000-character fallback and the 1,536-character entry cut (budget.client_budgets,docs/decisions/2026-09-30-skills-llm-native-listing.md); the manifest'sclaude_on_capis this repository's own ceiling, not a client budget.[skills] max_context_tokens(6,000 in the template) andcodex-rs/ext/skills/src/render.rsL19-22, L126-152 and L154-160 at rust-v0.159.2, as cited inbudget.client_budgets; the repository counterscripts/skills_status.pyestimates 3,171 tokens for the 28-skill catalog against 6,000.cliblock of the manifest, v1.7.0, https://github.com/vercel-labs/skills/tree/v1.7.0),skills add <tree URL> --skill claude-native-practice -g -y, as the other rows install.Evidence-class table
SKILL.mdhash, byte count and description length are computed from the landed commit's own bytes; the three sums equal the sums over the rows and stay within the ceilinglocal_integrationskills_row.py(lane tool, in the research record) andtests.test_skills_manifest(BudgetTests,SkillEntryTests)local_integrationscripts/skills_status.py --json: catalog matches the manifest, within budget (the host has not installed the skill yet, so its per-skill install states read missing, by design)local_integrationtests.test_skills_manifest.SkillEntryTests.test_audit_url_is_on_skills_shadoption_refcontract, the landscape catalog lists it once underagent-docs, and the Claude template's overrides equal the manifestlocal_integrationtests.test_runtime_worker_skills,tests.test_landscape_sweep_skills,tests.test_skills_manifest.ListingBudgetTemplateTestsDeclared test contract change
Three main test methods change by design:
tests.test_new_wsl_client_config.AgentGapTests.test_the_gaps_are_the_servers_and_skills_the_agents_name_beyond_the_wired_ones_and_the_plans_skills(the plan's selected skill count 27 to 28),tests.test_skills_status.SkillsStatusTests.test_real_manifest_codex_catalog_fits_the_configured_token_budget(the Codex catalog count 27 to 28; still must match the manifest total and fit the unchanged 6,000-token setting) andtests.test_skills_manifest.SkillEntryTests.test_audit_url_is_on_skills_sh(a first-party row is held to the repository source and the not-applicable audit form). Main's versions of these three fail against this head;docs/decisions/2026-10-10-skills-claude-native-practice-row.mddeclares them.Local commands run
12 files changed, 159 insertions(+), 35 deletions(-). The local full-suite shards in a CI-like environment are noisy on this host (four of eight red at a CI-green head on tests no change touches), so the list above is the targeted run and CI's eight shards are the full-suite measurement.
Decision record
docs/decisions/2026-10-10-skills-claude-native-practice-row.md: the ceiling raise and its cost, the listing check, what follows, the declared test changes, the inverse and the revisit.Host evidence
Not applicable: no file under
evidence/hosts/changes.Checklist
permissions: {}and grant each job only what it needs. (none changed)🤖 Generated with Claude Code