Repository navigation
feat(coding-agent): validate npm lifecycle and hosted runtime - #217
Merged
Merged
Conversation
- Protect installed updates and distribution contents with real package-manager acceptance and explicit public documentation selection. - Validate the compiled hosted runtime locally while registry publication and deployment promotion await configured environments.
- Match recovery paths to actual isolated backup directories so npm logging variations cannot block repair. - Reject unrelated, ambiguous, and symlinked paths before moving a retained backup.
- Preserve the interrupted CLI directory when npm cannot clean its partially installed dependencies. - Keep user data and unrelated global packages untouched during the pinned recovery.
- Keep project attribution in the consolidated third-party notices file. - Reject redundant NOTICE files in npm artifacts and hosted runtime layers.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Complete local npm distribution lifecycle validation and prepare the private hosted runtime for controlled candidate testing.
The companion cloud handoff is https://github.com/diffexai/diffex-cloud/pull/162.
Reasoning
Source updater tests did not prove real installed package-manager updates, and broad documentation copies could disclose future internal notes.
The owner confirmed npm-only distribution and that no staging deployment or Daytona consumer is configured.
Local runtime acceptance therefore uses the actual server process with mocked provider boundaries and external networking disabled.
GHCR publication, OCI attestations, real Daytona acceptance, promotion, and rollback remain pending configuration and their documented gates.
Changes
NOTICEgeneration and production copies; preserve consolidated attribution and reject reintroduced copies in tarballs and runtime layers.Testing
npm run checkand./test.shpassed.--network none, and every-layer inspection passed: 12 runtime workspaces and 30 approved documentation assets.The baseline CLI regression workflow passed.
The native artifact workflow passed on all four platforms using PR merge candidate
255001392a1fd131fb95b82d0f709323ed4b673d.Every runner accepted the same tarball, SHA-256
e4bff793cbe21e570596a552816ab465e15aee4bbc660b33aa332c664cdc8c1f, and recorded successful cleanup.macOS x64 reproduced npm's partial-dependency cleanup failure and passed the documented move-aside and pinned reinstall recovery.
The Runtime Image workflow passed on lifecycle commit
3e1a87fe4ff1769408a5927e8a2056f124175cc0and retained the tested Docker image, smoke result, layer inspection, and checksums privately.Public-to-public upgrades and real hosted inference are not established by controlled fixtures.
The NOTICE exclusion passed
npm run checkand all 34 packaging and runtime-layer regressions after reproducing four failing artifact-boundary assertions.The updated runtime validation passed on commit
59045f4a8e192c8db7d393386d3ab82d28b03e59, including the rebuilt image smoke and every-layer rejection of separate project NOTICE files.The updated native artifact acceptance checks the NOTICE-free npm candidate.