Skip to content

feat(manager): -payload-sha256 verifies the pinned wrapper-lite payload - #13

Merged
LYJW131 merged 1 commit into
litefrom
payload-sha256
Nov 3, 2026
Merged

LYJW131 merged 1 commit into
litefrom
payload-sha256

Conversation

@LYJW131

@LYJW131 LYJW131 commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Adds -payload-sha256 <hex> next to -payload-url. The zip is hashed before it is written; on a mismatch nothing is installed and the manager exits (an already installed payload is not kept). The digest is recorded in .payload-source, so a new digest reinstalls. Tests: match, mismatch, mismatch with an installed payload.

Lets amdl pin the lite payload by run id and sha256 instead of trusting the newest build-lite artifact.

🤖 Generated with Claude Code


Devin Review

…e pinned digest

A pinned -payload-url said where the wrapper-lite zip comes from but not
what it must contain. With -payload-sha256 the download is hashed before
it is written; a mismatch installs nothing and stops the manager even if
another payload is installed. The digest joins .payload-source, so a new
pin reinstalls.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Signed-off-by: LYJW131 <153256373+LYJW131@users.noreply.github.com>

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@LYJW131
LYJW131 merged commit 0345f26 into lite Nov 3, 2026
7 checks passed
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