[PR #70] [Parked] feat(snapshot): E2B alias rebuild semantics on template publish #155

New issue

Open

opened 2026-10-29 14:20:21 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-29 14:20:21 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/AgentENV/pull/70
Author: @JoyboyBrian
Created: 7/29/2026
Status: 🔄 Open

Base: main ← Head: e2b-build/02-alias-rebuild

📝 Commits (2)

9b9e271 fix(api): return the JSON error envelope for unmatched routes
f9e9c3e feat(snapshot): E2B alias rebuild semantics on template publish

📊 Changes

4 files changed (+456 additions, -100 deletions)

View changed files

📝 src/api/proxy.rs (+30 -1)
📝 src/snapshot/repository/backends/oss/repository.rs (+166 -35)
📝 src/snapshot/repository/backends/posixfs/backend.rs (+135 -10)
📝 src/snapshot/repository/backends/posixfs/catalog.rs (+125 -54)

📄 Description

Status: parked

This PR was moved to draft on August 6, 2026 (PT), following reviewer guidance that the alias system is expected to be refactored. It is intentionally not part of the active COPY/ADD stack and is not required by #71–#74.

The branch and discussion are being preserved rather than closed so the behavior and review history can be revisited after the alias refactor has a stable direction.

What this explored

E2B-style rebuild semantics for snapshot aliases:

keep the existing alias live while a replacement builds;
move the alias only after the new snapshot commits;
restore the prior alias/record state if publication fails;
keep the previous snapshot addressable by ID.

Why it is parked

The proposed behavior modifies alias binding, publication ordering, rollback, and previous-record cleanup. Landing that immediately before a broader alias-system refactor would create avoidable merge and migration work.

Existing-alias rebuild therefore remains unsupported for now. Callers should use a new alias or delete the existing template before rebuilding.

Active COPY/ADD stack

Layer	PR	Branch	Scope
1/4	#71	`e2b-build/03-build-file-store`	Build-context archive store
2/4	#72	`e2b-build/04-upload-api`	Upload API and configuration
3/4	#73	`e2b-build/05-copy-plan`	Host-side COPY planning
4/4	#74	`e2b-build/06-copy-exec`	COPY/ADD execution, E2E, and documentation

Those branches were replayed directly onto current main at 00ba6cb; none contains this PR.

Reference

Related: #28, which retains the original full-stack concept, design discussion, and earlier E2E evidence.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/AgentENV/pull/70 **Author:** [@JoyboyBrian](https://github.com/JoyboyBrian) **Created:** 7/29/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `e2b-build/02-alias-rebuild` --- ### 📝 Commits (2) - [`9b9e271`](https://github.com/kvcache-ai/AgentENV/commit/9b9e27194a78b261eecc27cb7074f9110920e67e) fix(api): return the JSON error envelope for unmatched routes - [`f9e9c3e`](https://github.com/kvcache-ai/AgentENV/commit/f9e9c3eb63a8865753bfbbfa0b35f7626bbb8623) feat(snapshot): E2B alias rebuild semantics on template publish ### 📊 Changes **4 files changed** (+456 additions, -100 deletions) <details> <summary>View changed files</summary> 📝 `src/api/proxy.rs` (+30 -1) 📝 `src/snapshot/repository/backends/oss/repository.rs` (+166 -35) 📝 `src/snapshot/repository/backends/posixfs/backend.rs` (+135 -10) 📝 `src/snapshot/repository/backends/posixfs/catalog.rs` (+125 -54) </details> ### 📄 Description ## Status: parked This PR was moved to draft on August 6, 2026 (PT), following reviewer guidance that the alias system is expected to be refactored. It is intentionally not part of the active COPY/ADD stack and is not required by #71–#74. The branch and discussion are being preserved rather than closed so the behavior and review history can be revisited after the alias refactor has a stable direction. ## What this explored E2B-style rebuild semantics for snapshot aliases: - keep the existing alias live while a replacement builds; - move the alias only after the new snapshot commits; - restore the prior alias/record state if publication fails; - keep the previous snapshot addressable by ID. ## Why it is parked The proposed behavior modifies alias binding, publication ordering, rollback, and previous-record cleanup. Landing that immediately before a broader alias-system refactor would create avoidable merge and migration work. Existing-alias rebuild therefore remains unsupported for now. Callers should use a new alias or delete the existing template before rebuilding. ## Active COPY/ADD stack | Layer | PR | Branch | Scope | |---|---|---|---| | 1/4 | #71 | `e2b-build/03-build-file-store` | Build-context archive store | | 2/4 | #72 | `e2b-build/04-upload-api` | Upload API and configuration | | 3/4 | #73 | `e2b-build/05-copy-plan` | Host-side COPY planning | | 4/4 | #74 | `e2b-build/06-copy-exec` | COPY/ADD execution, E2E, and documentation | Those branches were replayed directly onto current main at `00ba6cb`; none contains this PR. ## Reference Related: #28, which retains the original full-stack concept, design discussion, and earlier E2E evidence. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi added the

pull-request

label

2026-10-29 14:20:21 +00:00