Skip to content

fix(history): support Enterprise repositories and omp CLI host alias - #464

Merged
Helweg merged 6 commits into
mainfrom
fix/history-enterprise-and-omp-host
Oct 24, 2026
Merged

Helweg merged 6 commits into
mainfrom
fix/history-enterprise-and-omp-host

Conversation

@Helweg

@Helweg Helweg commented Oct 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • Support [HOST/]OWNER/REPO history sources on GitHub.com and GitHub Enterprise, routing authenticated gh REST/GraphQL requests to the selected host.
  • Preserve bare/explicit GitHub.com identity compatibility and valid projected v2 staging resume after fresh repository/principal/window proof. Keep caches, staging, citations, permission checks, and selected-source cleanup isolated across hosts, including identical repository IDs.
  • Accept CLI --host omp as the existing pi host/config/store; keep canonical host configuration unchanged. Update CLI/MCP guidance and documentation.
  • Give the cancellation-retirement regression test an existing active handler while asserting generation stability; retain its 130 cancellations, capacity/no-response checks, sleep/resume assertions, and original timeouts. This does not attribute the earlier untraced PID failure.

Verification

Qualified commit: 1df0f491ae30c80b9d335468d47080e984cbfbba.

  • Build, typecheck, and lint passed.
  • Canonical unfiltered suite with pretest:run intact: 2911 passed, 0 failed, 6 skipped.
  • 19 Enterprise history cases and 13 history-surface cases, including the OMP/Pi CLI alias; the full 17-case supervisor file also passed after the test-setup repair.
  • Real built CLI and MCP stdio fixture probes passed sync/search/status/clear, selected-host citations, cross-host isolation, and OMP/Pi config/store behavior; MCP exited cleanly without protocol errors.

Enterprise behavior was exercised with owned deterministic gh fixtures, not a live GitHub Enterprise instance. No version bump or release is included.

@Helweg Helweg added bug Bug fix semver:patch Bug fix or patch labels Oct 23, 2026
@Helweg

Helweg commented Oct 24, 2026

Copy link
Copy Markdown
Owner Author

Automated PR review

Reviewed head 1df0f491ae30c80b9d335468d47080e984cbfbba against base 6b7c61e41c91584b12e7908e283d940184f4d2c3.

No actionable findings in the reviewed diff

Checked all 27 changed files and relevant surrounding callers: repository-selector normalization, selected-server REST/GraphQL routing, capability/principal checks, cache and staging isolation, legacy staging/snapshot compatibility, citations and direct-reference search, selected-source cleanup, the omp-to-Pi CLI alias, adapter guidance, docs/changelog, and the cancellation-retirement regression-test change. Complete source comparisons matched the changed-file diff counts. No existing review discussions were present when checked.

Observed CI: all 19 checks returned for this head completed successfully, including Node.js 22/24 test jobs, the path-semantics matrix, and CodeQL. The test jobs report successful build, typecheck, lint, and coverage-test steps (Node.js 22, Node.js 24).

Verification limits: static review and observed CI only; candidate code/tests were not executed. Live GitHub Enterprise behavior was not independently exercised; the PR describes deterministic fixture-based Enterprise verification. This comment is not an approval or merge decision.

@Helweg Helweg added the orca-reviewed Orca posted a review; its comment identifies the reviewed head/base revisions. label Oct 24, 2026
@Helweg
Helweg merged commit c154892 into main Oct 24, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix orca-reviewed Orca posted a review; its comment identifies the reviewed head/base revisions. semver:patch Bug fix or patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant