Skip to content

Add safe local extension deployment and health checks - #54

Merged
wolfiesch merged 3 commits into
mainfrom
iteration-tooling-pr
Sep 19, 2026
Merged

wolfiesch merged 3 commits into
mainfrom
iteration-tooling-pr

Conversation

@wolfiesch

@wolfiesch wolfiesch commented Sep 19, 2026

Copy link
Copy Markdown
Owner

Summary

Verification

Deployment safety and rollback regressions, CLI contracts, and workspace typechecks passed. SDK build passed. Earlier combined-runtime live verification remains historical; this narrowed head is verified by its focused checks.

@omp-maintainer omp-maintainer 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.

P3 — four independent feature/fix threads across 23 files prevent merge-at-glance.
Blocking: the deploy receipt follows symlinks and can overwrite arbitrary user files; the documented POSIX workflow uses a macOS-only default reload.
Should-fix: sanitize selector text in OMP rendering, make the TypeScript retry test prove a retry, and use Bun's crypto API.
Maintainer call: split deployment/doctor, selector actions, and reconnect/reconciliation into focused PRs. Thanks for the detailed implementation and verification notes.

Comment thread scripts/dev_deploy.py
Comment thread docs/setup.md
Comment thread packages/omp/src/render.ts Outdated
Comment thread packages/sdk-typescript/test/client.test.ts Outdated
Comment thread packages/installer/test/cli.test.ts Outdated
@wolfiesch wolfiesch changed the title Improve local extension iteration and reconnect safety Add safe local extension deployment and health checks Sep 19, 2026
@wolfiesch
wolfiesch merged commit d8bb763 into main Sep 19, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant