Skip to content

fix(package): bind prepared and cross-target evidence to the actual graph - #163

Draft
gael-obin wants to merge 2 commits into
codefly-dev:fix/honour-prepared-goworkfrom
gael-obin:fix/prepared-workspace-evidence
Draft

gael-obin wants to merge 2 commits into
codefly-dev:fix/honour-prepared-goworkfrom
gael-obin:fix/prepared-workspace-evidence

Conversation

@gael-obin

Copy link
Copy Markdown

Prepared-workspace packaging could emit evidence from a different dependency graph than the executable. This owner fix keeps native inventory on the prepared workspace and cross-target inventory on its pinned graph, and strengthens public Package regressions to validate source attachment, linked sibling modules, graph edges and executable digest. It reuses Core inventory and artifact attachment; no second scanner is introduced.

This is a follow-up to #159, targeting that existing branch. The current account cannot push to its owner branch, so the reviewed correction is supplied from a fork without rewriting its history. Core v0.19.10 adoption retains the existing ownership-writer path; it is not a release of this development agent.

BLOCKED pending the owner release/adoption of codefly-dev/core#878. The stronger test exposes a real existing Core multi-main inventory defect with the committed dependency. With an uncommitted workspace selecting reviewed Core2b077f2e, full service-go suite/build/vet and real native Darwin/arm64 + cross Linux/amd64 CGO packaging pass. Core source review closed root, graph and alternate-manifest defects. No committed pseudo-version or replace bypass is proposed.

Normal CLI0.1.233 bootstraps its packager with the committed Core pin, so local final-executable substitution alone still emits incomplete evidence. Repeat CLI packaging and clean/generated delivery after released adoption. Linux process-lifecycle qualification remains open; cross-compilation is not Linux execution. Exact reviewed source60d721eddbc62de4a069b740fc83612b5c0ed068. Keep draft; do not merge this dependent follow-up as-is.

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.

1 participant