Skip to content

Pin vendor skills for CC lane messaging, evals and logs - #852

Merged
seathatflowsinourveins merged 6 commits into
mainfrom
codex/skills-cc-lanes-20261008
Oct 8, 2026
Merged

seathatflowsinourveins merged 6 commits into
mainfrom
codex/skills-cc-lanes-20261008

Conversation

@seathatflowsinourveins

@seathatflowsinourveins seathatflowsinourveins commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Scope

Pin the maintainers' hcom-agent-messaging, promptfoo-evals and loki skills for command-center and native lane work. Their descriptions add 890 characters; the repository counter reports Claude 10,374/10,500 and Codex 10,055. Keep promql deferred to the owner's cap decision and keep the runtime-worker selection at 134 skills with explicit exclusions for these three lane picks.

The decision compares each vendor workflow with native documentation/operation routes and maintained community definitions. Promptfoo's skill uses the supported profile's 0.123.1 source revision; its entire folder is identical at 0.124.0, so the correction preserves the instructions and listing totals.

  • Base commit: 28af9b9321d0492f6cdabe75091376560e7cdc4d.
  • Lane: lane:foundation.
  • Owned paths: the central skill manifest, both listing templates, worker skill manifest/README, lifecycle source/task catalog, refreshed foundation citation, renderer and affected tests, dated decision, and evidence registry hashes.

When loki appears in the native Claude skillOverrides map, the renderer's service-name check treats that key as a skill identifier. Actual Loki endpoints, MCP server references and override values remain checked for missing service wiring. The scan does not change the rendered configuration or declare Loki wired.

SOTA sources

Evidence-class table

Claim Evidence class Command / record
Vendor bodies, folder trees, byte hashes, licenses and descriptions at immutable pins source_review The decision record and central manifest
Claude sum 10,374 fits 10,500; Codex sum 10,055 and 27 catalog skills local_integration Unchanged scripts/skills_status.py:budget_report
Catalogs, counts and worker central-adoption contract agree with the reviewed source pins local_integration Four affected native unittest modules: 232 run, 2 skipped, 0 failures
Skill identifiers are separate from service references; actual unwired references still fail local_integration Render-boundary regression failed before the fix; endpoint/MCP/value controls and existing render checks pass
Registered artifacts and repository scope validate local_integration python3 scripts/validate.py: exit 0

Local commands run

python3 -m unittest tests.test_skills_manifest tests.test_skills_status tests.test_landscape_sweep_skills tests.test_runtime_worker_skills
exit 0: 232 tests run, 2 skipped, 0 failures

python3 scripts/validate.py
exit 0: status passed; 69 components, 10,822 hashed files, 4 profiles, 226 receipts

git diff --check
exit 0

The existing renderer regression and endpoint/MCP/value controls also pass (four focused native methods). The decision retains failed conditions from initial local and hosted validation, including the source/task projections and the skill/service namespace collision. Marketplace scan verdicts remain Unknown; their links are locators.

Decision record

docs/decisions/2026-10-08-skills-cc-lanes-picks.md records the scoped source comparison, supported Promptfoo revision, counters, source-review correction, worker exclusions, inverse and per-selection overturn conditions. Applicability is source evidence; comparative outcomes and client invocation acceptance remain unmeasured.

Host evidence

Installation and invocation are deferred to the designated host shell outside Claude after this PR lands (CC plan row 31). This PR records no new skill use, evaluation, component install or live client settings change. The paper-runtime freeze and quiet window remain binding.

Checklist

  • Vendor definitions are referenced at immutable commits; no body is vendored or rewritten.
  • No GitHub Actions workflow changed.
  • No secret value or private native transcript is committed.
  • No paid hosting, subscription or billing surface introduced.
  • Peer files and worktrees preserved.

@seathatflowsinourveins seathatflowsinourveins added the lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers label Oct 8, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T09:03:19.439192Z 0a243fc PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@socket-security

socket-security Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0a243fcadf

ℹ️ 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".

Comment thread adoption/skills/manifest.json Resolved
Comment thread adoption/skills/manifest.json Resolved
@seathatflowsinourveins
seathatflowsinourveins force-pushed the codex/skills-cc-lanes-20261008 branch from 1694009 to ab5280c Compare October 8, 2026 12:39
@seathatflowsinourveins
seathatflowsinourveins merged commit 491acee into main Oct 8, 2026
35 of 36 checks passed
@seathatflowsinourveins
seathatflowsinourveins deleted the codex/skills-cc-lanes-20261008 branch October 8, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant