fix(viewer): restore live updates and sibling assets#474
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis PR expands content token signing to control path scoping, collects multiple sibling file paths in revision URLs, integrates signed URLs into the web route and frontend viewer hooks, and enriches live-update callbacks with revision metadata. Durable object bindings are added to environment-specific Wrangler configuration. ChangesSigned viewer URL flow
Durable objects binding configuration
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
agent-paste PR preview resources were cleaned up. The shared Preview GitHub Environment is retained for future preview deploys. |
Summary
Production needs the local viewer fixes shipped through CI. This restores live viewer updates, keeps the iframe mounted through transient metadata refetch failures, and fixes signed content URLs so same-revision sibling assets can load without loosening web viewer scope beyond the artifact file list.
Changes
ARTIFACT_LIVEDurable Object binding to preview and production Wrangler envs.Risk: HIGH
getAgentViewlookup when serving web artifact details with a viewer.Test plan
pnpm verifypnpm test:coverageart_0TA3F2YKDMKRQZDJ3NGEW1G50Z; already-open preview viewer updated without reload; iframe HTML andtest-image.pngreturned 200; no browser console errors.Linear: none
Summary by CodeRabbit
New Features
Bug Fixes