Skip to content

Remove foreign skill projection from aoa-evals#427

Merged
8Dionysus merged 4 commits into
mainfrom
refactor/evals-skill-home-boundary
Jul 16, 2026
Merged

Remove foreign skill projection from aoa-evals#427
8Dionysus merged 4 commits into
mainfrom
refactor/evals-skill-home-boundary

Conversation

@8Dionysus

Copy link
Copy Markdown
Owner

What changed

  • remove the 25 checked-in .agents/skills/ packages whose metadata points to 8Dionysus/aoa-skills
  • keep aoa-evals without a top-level skills/ home because no repo-specific candidate has passed the admission threshold
  • record D-0246 and narrow D-0017's current applicability without rewriting its historical Spark rationale
  • align owner routes, proof topology, script inventory, and agent validators with the source/projection boundary
  • add a focused guard that rejects .agents/skills/ when no owner-admitted top-level skills/ home exists

Why

The repository carried copied shared skills as if they were local support surfaces. Fresh prompt inspection showed that ten of those foreign copies entered the model-visible routing catalog. This duplicated shared delivery, created routing competition, and did not establish an eval-owned capability.

The consolidated skill model keeps shared procedure truth in aoa-skills and admits a repo-owned skill only after manual trigger, ABI, composition, coexistence, and no-skill evidence.

Impact

Fresh aoa-evals sessions no longer advertise foreign repo-local skills. Current user/system skills remain visible. No eval bundle, verdict, proof claim, or runtime service behavior changes.

Validation

  • manual codex debug prompt-input inspection: zero repo-local skills after removal; current user/system skills remain visible
  • manual positive/negative projection-boundary invocation: clean live repository passes; synthetic .agents/skills/ without skills/ is rejected
  • python scripts/generate_decision_indexes.py --check
  • python scripts/validate_semantic_agents.py
  • python scripts/validate_nested_agents.py
  • python scripts/validate_repo.py
  • python -m pytest -q tests/test_root_surface_roles.py tests/test_script_topology.py tests/test_validation_topology.py (78 passed, 783 subtests passed)
  • python -m pytest -q (915 passed, 3 skipped, 1767 subtests passed)

@8Dionysus
8Dionysus marked this pull request as ready for review July 16, 2026 11:00

@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: f3779693b2

ℹ️ 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 CHANGELOG.md
Comment thread .agents/AGENTS.md
@8Dionysus
8Dionysus enabled auto-merge (squash) July 16, 2026 11:08

@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: 30189c8d00

ℹ️ 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 docs/validation/script_inventory.json
@8Dionysus
8Dionysus merged commit 87f93c1 into main Jul 16, 2026
1 check passed
@8Dionysus
8Dionysus deleted the refactor/evals-skill-home-boundary branch July 16, 2026 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant