Skip to content

fix: publish repository targets under canonical package names - #134

Merged
ORESoftware merged 5 commits into
mainfrom
agent/publish-canonical-polyglot-package
Aug 4, 2026
Merged

fix: publish repository targets under canonical package names#134
ORESoftware merged 5 commits into
mainfrom
agent/publish-canonical-polyglot-package

Conversation

@ORESoftware

@ORESoftwareORESoftware commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What changed

  • preserve the existing one-artifact-per-declared-target fan-out contract
  • treat an explicit target with dir = "." as the canonical whole-repository package
  • emit that artifact under the root manifest's exact org/name, even when an older manifest calls the target package *-repository
  • continue re-rooting and isolating every language/schema target
  • add focused unit coverage for canonical identity, full-repository contents, deterministic artifact naming, target isolation, and pack-output exclusion

Why

The interface repositories already declare [package].name correctly and already carry a [targets.repository] rooted at .. Before this patch, zed pack/publish renamed that artifact to *-repository, so zed install org/<repository-name> could not resolve the full interface package. Canonicalizing only the explicit root target fixes the requested packages without changing target-only manifests or the established pack_all cardinality.

Compatibility

  • manifests without targets still publish one root package
  • target-only polyglot manifests still publish exactly their declared target packages
  • manifests with [targets.repository] dir = "." publish that target as the exact root package plus all language/schema targets

Checks

  • focused Rust unit tests are included in src/pack.rs
  • the earlier broad e2e rewrite was removed from branch history; this PR now changes only src/pack.rs
  • full repository CI is running on the current head

@ORESoftware
ORESoftwareforce-pushed the agent/publish-canonical-polyglot-package branch from 1de9d88 to 5cffe3aCompareAugust 4, 2026 16:43
@ORESoftwareORESoftware changed the title fix: publish canonical package for polyglot repositoriesfix: publish repository targets under canonical package namesAug 4, 2026

@ORESoftwareORESoftware left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Manual semantic review of final head 2303e69a:

  • pack_all retains one artifact per declared target for polyglot manifests.
  • only dir = "." targets have their derived package name reset to the root manifest name.
  • nested-target manifest rejection remains intact.
  • full-repository staging still uses the existing exclusion and deterministic archive paths.
  • language target re-rooting and adapter/ecosystem derivation remain unchanged.
  • focused tests cover exact artifact name, full-tree contents, output-directory exclusion, and language isolation.

Hosted Actions jobs are queued rather than reporting a code failure.

@ORESoftware
ORESoftware merged commit 5cef340 into mainAug 4, 2026
19 of 22 checks passed
@linear-code

Copy link
Copy Markdown

DEN-1837

ORESoftware added a commit that referenced this pull request Aug 8, 2026
Add a trusted root `zed gitops` dispatcher that executes only the sibling `zed-gitops` binary, rejects ambient PATH, recursion, and missing/non-executable commands, propagates arguments and exit status without shell interpretation, and updates help, completions, package outputs, documentation, and focused tests.
Exact head passed all 21 product workflows and independent Ubuntu/macOS/Windows plus full-lifecycle certification in zed-pkg-test #134.
Sign up for freeto 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

@ORESoftware