Skip to content

Build hierarchical portable tool catalog - #85

Closed
omry wants to merge 1 commit into
pr83from
pr84
Closed

Build hierarchical portable tool catalog#85
omry wants to merge 1 commit into
pr83from
pr84

Conversation

@omry

@omry omry commented Aug 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Load strict content-addressed Java and Playwright records from the embedded hierarchical catalog. Resolve exact releases, targets, bindings, selections, package sets, and acquisition sources into a selected closure.

Validate namespaces, reference digests, resource limits, artifact source mappings, target fixtures, and immutable artifact inventories. Add catalog and negative-path coverage and align the portable-tool design with the implemented layout.

@omry
omry changed the base branch from main to pr83 August 16, 2026 05:04
@omry
omry marked this pull request as ready for review August 16, 2026 05:18
Copilot AI lite review requested due to automatic review settings August 16, 2026 05:18
Copilot AI reviewed Aug 16, 2026

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@omry

omry commented Aug 16, 2026

Copy link
Copy Markdown
Owner Author

@codex review 95286c1

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 16, 2026

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 95286c1b6e

ℹ️ 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".

@omry omry added approved PR reviewed and approved and removed approved PR reviewed and approved labels Aug 16, 2026
@omry
omry force-pushed the pr83 branch 2 times, most recently from 92bc560 to 272fcf1 Compare August 16, 2026 06:17
@omry
omry force-pushed the pr84 branch 2 times, most recently from 5ef24af to 115b269 Compare August 16, 2026 06:24
@omry
omry force-pushed the pr83 branch 2 times, most recently from 6cf3dd1 to 77d8655 Compare August 16, 2026 06:37
@omry
omry force-pushed the pr84 branch 2 times, most recently from 0c15e3a to 8eeb9e5 Compare August 16, 2026 06:44
@omry
omry force-pushed the pr83 branch 2 times, most recently from 2442f45 to b86c62b Compare August 16, 2026 06:59
@omry
omry force-pushed the pr84 branch 2 times, most recently from 0f8e531 to dada6a4 Compare August 16, 2026 07:18
@omry
omry force-pushed the pr83 branch 2 times, most recently from 8285147 to d0270fb Compare August 16, 2026 08:22
@omry
omry force-pushed the pr84 branch 2 times, most recently from 7addad3 to 9d70850 Compare August 16, 2026 13:41
@omry
omry force-pushed the pr83 branch 2 times, most recently from 9d15f10 to 836663c Compare August 16, 2026 13:56
@omry
omry force-pushed the pr84 branch 2 times, most recently from a628f86 to a9846f1 Compare August 16, 2026 14:09
@omry
omry force-pushed the pr83 branch 2 times, most recently from 2682373 to 5c87e89 Compare August 16, 2026 14:19
@omry
omry force-pushed the pr84 branch 2 times, most recently from 9131cb7 to a0befa5 Compare August 16, 2026 14:39
@omry
omry force-pushed the pr83 branch 2 times, most recently from 29d1dc9 to 509ba56 Compare August 16, 2026 14:47
@omry
omry force-pushed the pr84 branch 2 times, most recently from c073c4c to 9eced28 Compare August 16, 2026 15:43
@omry
Build hierarchical portable tool catalog
00dafa7 
Load strict content-addressed Java and Playwright records from the embedded hierarchical catalog. Resolve exact releases, targets, bindings, selections, package sets, and acquisition sources into a selected closure.

Validate namespaces, reference digests, resource limits, artifact source mappings, target fixtures, and immutable artifact inventories. Add catalog and negative-path coverage and align the portable-tool design with the implemented layout.
@omry

omry commented Aug 17, 2026

Copy link
Copy Markdown
Owner Author

Superseded by the portable-tool reconstruction. This PR's content is being split into individually reviewable slices PTD-07 through PTD-11 per docs/PORTABLE_TOOL_DEFINITION_IMPLEMENTATION_PLAN.md, each landing as its own commit and PR.

No content is lost. Local extraction source: 37ca781bd6cb (rewritten from b680caaa834e).

1 similar comment
@omry

omry commented Aug 17, 2026

Copy link
Copy Markdown
Owner Author

Superseded by the portable-tool reconstruction. This PR's content is being split into individually reviewable slices PTD-07 through PTD-11 per docs/PORTABLE_TOOL_DEFINITION_IMPLEMENTATION_PLAN.md, each landing as its own commit and PR.

No content is lost. Local extraction source: 37ca781bd6cb (rewritten from b680caaa834e).

@omry omry mentioned this pull request Aug 17, 2026
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@omry