Skip to content

Resolve registered primary paths to managed repo identities in fanout - #12002

Closed
chubes4 wants to merge 3 commits into
mainfrom
fix/11984-primary-path-resolution-v2
Closed

Resolve registered primary paths to managed repo identities in fanout#12002
chubes4 wants to merge 3 commits into
mainfrom
fix/11984-primary-path-resolution-v2

Conversation

@chubes4

@chubes4 chubes4 commented Aug 8, 2026

Copy link
Copy Markdown
Member

Summary

  • resolve an exact registered primary checkout path to its managed component slug before fanout child planning
  • preserve repository slugs unchanged while rejecting related worktrees, nested paths, ambiguous registrations, and unregistered paths with structured guidance
  • expose the owning-layer primary-path resolution primitive from homeboy-core and document the accepted --repo forms
  • verify normalization at both the core boundary and the fanout command boundary

Verification

  • cargo fmt --all -- --check
  • cargo test --locked -p homeboy-core registered_primary_path_resolution_is_exact_and_canonical --lib
  • cargo test --locked -p homeboy-core registered_primary_path_resolution_rejects_related_and_ambiguous_checkouts --lib
  • cargo test --locked -p homeboy-cli cook_batch_repo_normalization_accepts_slugs_and_registered_primary_paths --lib
  • final independent review reported no findings

Closes #11984.

AI Assistance

OpenAI gpt-5.6-sol via OpenCode analyzed the primary-path failure, implemented the shared resolver and fanout integration, and reviewed and verified the patch under human direction. Chris Huber remains responsible for every line.

@homeboy-ci

homeboy-ci Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Homeboy Results — homeboy

Lint

review lint — passed

ℹ️ Full options: homeboy self docs commands/lint
ℹ️ Save lint baseline: homeboy review lint homeboy --baseline
Deep dive: homeboy review lint homeboy --changed-since 66a6691

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-homeboy-review-lint-homeboy-fast-Linux contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-homeboy-review-lint-homeboy-fast-Linux contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/homeboy/actions/runs/31288158373

Audit

⚠️ review audit — no measurement

Deep dive: homeboy review audit homeboy --changed-since 66a6691

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-homeboy-review-audit-homeboy-fast-Linux contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-homeboy-review-audit-homeboy-fast-Linux contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/homeboy/actions/runs/31288158373
Tooling versions
  • Homeboy CLI: homeboy 0.334.0+ba85b77bd72a
  • Extension: rust from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: a32e4ea3
  • Action: unknown@unknown

@chubes4

chubes4 commented Aug 9, 2026

Copy link
Copy Markdown
Member Author

CI prerequisite update: this branch now merges #12001 non-destructively because current main cannot compile the workspace test suite without its AdoptArgs fixture repair. The resolver commit remains independent; once #12001 lands, this PR diff reduces to #11984 plus the merge topology.

AI assistance: OpenAI gpt-5.6-sol via OpenCode identified the prerequisite and performed the non-destructive branch update under human direction. Chris Huber remains responsible for the change.

@chubes4

chubes4 commented Aug 9, 2026

Copy link
Copy Markdown
Member Author

Superseded by #12012, which landed registered primary-path resolution on main. AI assistance disclosure: OpenAI gpt-5.6-sol via OpenCode was used to verify patch equivalence and reconcile the PR queue.

@chubes4 chubes4 closed this Aug 9, 2026
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.

Resolve registered primary paths to managed repo identities in fanout

1 participant