Skip to content

Record deployment lock (core PR #66) - #1

Merged
mbreissi merged 1 commit into
mainfrom
record/deployment-lock
Jul 23, 2026
Merged

Record deployment lock (core PR #66)#1
mbreissi merged 1 commit into
mainfrom
record/deployment-lock

Conversation

@mbreissi

Copy link
Copy Markdown
Contributor

Design-record update for deployment lock, shipped in edgecommons/edgecommons#66.

  • PLAN.md — the lock is recorded alongside the Greengrass renderer: what it resolves, that it retires the artifact.greengrassName override, that the §8.5.5 compatibility guard is now wired end to end, and that degradation is stated (EC4006/EC5006/EC5007) rather than implied.
  • design/REVIEW.md — decision #15's "until deployment lock resolves it" framing is replaced: the override is now optional (override first, lock second) rather than required.

No code here; this repo is the design surface.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LsX7cWVGLszurNYMVwMJGo

…and stated degradation

core PR #66. Three things the deck now depends on being true:

- `deployment lock` writes `<stem>.lock` beside the definition, so
  validate/render/plan are pure functions over files already in Git.
- The lock retires `artifact.greengrassName`: the renderer resolves the
  name override-first, then from the lock, and still refuses with neither.
- The §8.5.5 compatibility guard is wired end to end and starts enforcing
  by itself once components publish config schemas (RM-013).

REVIEW.md's #15 entry drops the 'until then' framing — the override is now
optional rather than required.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LsX7cWVGLszurNYMVwMJGo
@mbreissi
mbreissi marked this pull request as ready for review July 23, 2026 16:58
@mbreissi
mbreissi merged commit cedb484 into main Jul 23, 2026
1 check passed
@mbreissi
mbreissi deleted the record/deployment-lock branch July 23, 2026 16:58
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