Skip to content

core-skills: bind loaded skill identity to environment paths#25125

Draft
starr-openai wants to merge 3 commits into
starr/skills-path-ref-1b-bound-canonicalizefrom
starr/skills-path-ref-2-skill-identity
Draft

core-skills: bind loaded skill identity to environment paths#25125
starr-openai wants to merge 3 commits into
starr/skills-path-ref-1b-bound-canonicalizefrom
starr/skills-path-ref-2-skill-identity

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

@starr-openai starr-openai commented May 29, 2026 •
edited
Loading

Summary

  • bind loaded skill source identity to EnvironmentPathRef
  • keep raw skill paths for display, wire payloads, and config selectors while reading and deduping by bound source path
  • preserve filesystem authority in disabled paths, implicit indexes, root dedupe, and skill caches

Stack

Validation

  • cd /Users/starr/code/codex-worktrees/pr-25098-restack4/codex-rs && just fmt
  • cd /Users/starr/code/codex-worktrees/pr-25098-restack4/codex-rs && just test -p codex-core-skills
  • GitHub CI pending on rewritten head

@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from b499b0b to 1e031d3 Compare May 29, 2026 18:14
@starr-openai starr-openai changed the title core-skills: bind skill identities to environment paths core-skills: bind skill roots to environment paths May 29, 2026
@starr-openai starr-openai mentioned this pull request May 29, 2026
Draft
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from 0f9a588 to 6c5eb2f Compare May 29, 2026 18:19
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from 1e031d3 to 09e2db3 Compare May 29, 2026 18:19
@starr-openai starr-openai changed the title core-skills: bind skill roots to environment paths core-skills: bind skills to environment paths May 29, 2026
@starr-openai starr-openai mentioned this pull request May 29, 2026
Closed
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from 6c5eb2f to cd65f48 Compare May 29, 2026 18:24
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch 3 times, most recently from 90a5d1c to f64a138 Compare May 29, 2026 18:30
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from cd65f48 to df5d715 Compare May 29, 2026 18:30
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from f64a138 to d3e8d0f Compare May 29, 2026 18:41
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch 2 times, most recently from c4e1e54 to dd9cff2 Compare May 29, 2026 18:44
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch 2 times, most recently from 934ed76 to 3726074 Compare May 29, 2026 19:19
@starr-openai starr-openai changed the title core-skills: bind skills to environment paths core-skills: bind loaded skill identity to environment paths May 29, 2026
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch 2 times, most recently from 309b917 to 9d2a865 Compare May 29, 2026 19:57
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from dd9cff2 to eb0bf0b Compare May 29, 2026 20:22
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch 2 times, most recently from 2bdfde9 to b862ae1 Compare May 29, 2026 20:24
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from eb0bf0b to 2fdb7c5 Compare May 29, 2026 20:24
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from b862ae1 to 305d9e4 Compare May 29, 2026 20:24
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from 2fdb7c5 to d3938c8 Compare May 29, 2026 20:41
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from 305d9e4 to 3487328 Compare May 29, 2026 20:41
@starr-openai starr-openai changed the base branch from starr/skills-path-ref-1-env-path-ref to starr/skills-path-ref-1b-bound-canonicalize May 29, 2026 20:41
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1b-bound-canonicalize branch from ab25267 to 6fd29a0 Compare May 29, 2026 21:31
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from 3487328 to 4e3847a Compare May 29, 2026 21:31
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from 4e3847a to d2a9523 Compare May 29, 2026 21:32
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1b-bound-canonicalize branch from 6fd29a0 to 865271a Compare May 29, 2026 21:51
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from d2a9523 to c6e8026 Compare May 29, 2026 21:52
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1b-bound-canonicalize branch from 865271a to 1170d1c Compare May 29, 2026 22:14
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch 2 times, most recently from 01ff1d1 to 2e14910 Compare May 29, 2026 22:29
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1b-bound-canonicalize branch 2 times, most recently from 1119f59 to 641000f Compare May 29, 2026 22:44
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from 2e14910 to e9b615e Compare May 29, 2026 22:44
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1b-bound-canonicalize branch from 641000f to 8c24577 Compare May 29, 2026 22:50
@starr-openai starr-openai force-pushed the starr/skills-path-ref-2-skill-identity branch from e9b615e to 033d1d0 Compare May 29, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@starr-openai