Skip to content

Move the diagnostic skill into its owner home - #313

Merged
8Dionysus merged 2 commits into
mainfrom
codex/abyss-owner-skill-home-20260717
Jul 18, 2026
Merged

Move the diagnostic skill into its owner home#313
8Dionysus merged 2 commits into
mainfrom
codex/abyss-owner-skill-home-20260717

Conversation

@8Dionysus

Copy link
Copy Markdown
Owner

What changed

  • establishes skills/abyss-self-diagnostic-spine/ as the canonical owner package in abyss-stack, with a focused SKILL.md, typed contract, progressively disclosed diagnosis procedure, and OpenAI presentation metadata
  • retires the duplicate .agents/skills/abyss-self-diagnostic-spine overlay while preserving the owner-to-global projection route through skills/port.manifest.json
  • aligns the diagnostic-spine documentation, decisions, and existing validation modules with the new owner boundary
  • refreshes the repository-local portable KAG family so the skill, its full contract, its procedure, and the owner decision are source-returnable with provenance

Why

The previous overlay exposed a short instruction but did not give the runtime capability a canonical owner home or a complete operational contract. The new package preserves the original diagnosis function while making applicability, input/output ABI, freedom, effects, verification, termination, failure modes, provenance, relations, and lifecycle health explicit. It is designed to be globally advertised by the OS profile after merge, while abyss-stack remains the source of truth.

Manual behavioral evidence

The candidate was exercised in fresh Codex sessions before promotion, including:

  • natural positive routing without naming the skill
  • nearby generic-health negative routing
  • invalid-packet rejection
  • read-only observation
  • explicit bounded capture with exactly one requested write
  • stale-packet rejection
  • staged coexistence with aoa-session-recovery, loaded only after a verified repair handoff
  • return-to-source iterations that removed duplicate packet reads, schema hunting for review-only fields, unbounded array output, and an unnecessary heredoc attempt

No session trace, temporary fixture, or trial-only validator is committed.

Validation

  • skill-creator/scripts/quick_validate.py skills/abyss-self-diagnostic-spine
  • full owner release lane: 843 tests passed, 1 skipped, 144 subtests passed; synthetic source/runtime parity passed
  • existing shellcheck lane passed
  • python scripts/validate_stack.py
  • decision indexes and decision graph validation, including a temporary assembled graph
  • repo-local KAG full and incremental parity checks against the PR base
  • repo-local KAG family validation: 1,345 artifacts, 15,521 anchors, 6,535 entities, 804 events, 2,888 assertions, and 12,030 relations
  • deterministic seven-file compatibility assembly
  • live KAG queries returned the diagnosis procedure and typed contract first for a natural diagnostic request, with owner-return provenance

The KAG refresh touches many content-addressed shards because changed owner documents disperse anchors across shard hashes. The repository-standard one-change budget receipt records that reviewed generated delta; the standing budget was not raised.

@8Dionysus
8Dionysus marked this pull request as ready for review July 18, 2026 00:22
@8Dionysus
8Dionysus merged commit c6aafb0 into main Jul 18, 2026
4 checks passed
@8Dionysus
8Dionysus deleted the codex/abyss-owner-skill-home-20260717 branch July 18, 2026 00:22

@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: 995b150559

ℹ️ 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 on lines +266 to +268
- `aoa-session-recovery` in its explicit `diagnose` or `repair` mode
- `aoa-session-progression-lift`
- `aoa-quest-harvest`
- `aoa-session-harvest` in its explicit promotion mode

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep diagnostic handoff targets schema-compatible

This handoff list now points agents at aoa-session-recovery and aoa-session-harvest, but the typed diagnostic surfaces still accept and emit the older targets: repair_handoff.schema.json/aoa_diagnose.py require aoa-session-self-repair, and the companion/review validators still require aoa-session-self-diagnose/aoa-quest-harvest. When an agent follows this new owner-skill guidance for a repair or harvest handoff, it will produce a route that the owner packets cannot validate or expose; keep these names aligned with the schemas/code or update the schemas, generator, and projection together.

Useful? React with 👍 / 👎.

Comment thread skills/port.manifest.json
"bundles": [
{
"name": "abyss-self-diagnostic-spine",
"path": "skills/abyss-self-diagnostic-spine",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Include admitted skills in Configs sync

This bundle is now advertised from skills/abyss-self-diagnostic-spine, but the source-to-runtime projection still omits the entire skills tree (managed_items in aoa_sync_configs.sh and SYNC_MANAGED_ITEMS only include compose/docs/mechanics/mcp/quests/scripts/schemas/stats/etc.). After a normal aoa-sync-configs, the deployed Configs mirror has no skills/port.manifest.json or bundle for profile/source-return resolution, and parity validation cannot catch the omission; add skills to the sync allowlists and mirror checks with this admission.

Useful? React with 👍 / 👎.

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