Skip to content

Add just dev to start all services for local development#7

Merged
intendednull merged 3 commits into
mainfrom
claude/cli-deploy-services-6HOqy
Mar 29, 2026
Merged

Add just dev to start all services for local development#7
intendednull merged 3 commits into
mainfrom
claude/cli-deploy-services-6HOqy

Conversation

@intendednull
Copy link
Copy Markdown
Owner

Adds scripts/dev.sh which orchestrates the full local stack:
relay, replay worker, storage worker, and web UI (trunk serve).
Uses persistent identity keys in .dev/ so peer IDs are stable
across restarts. Colored log prefixes per service, graceful
shutdown on Ctrl+C.

https://claude.ai/code/session_01NPLijSpjCg48htC37k2AkM

claude added 3 commits March 29, 2026 10:55
Adds scripts/dev.sh which orchestrates the full local stack:
relay, replay worker, storage worker, and web UI (trunk serve).
Uses persistent identity keys in .dev/ so peer IDs are stable
across restarts. Colored log prefixes per service, graceful
shutdown on Ctrl+C.

https://claude.ai/code/session_01NPLijSpjCg48htC37k2AkM
Adds `just dev` / `just dev-quick` / `just dev-clean` to the command
reference and a new "Local Development Stack" section explaining the
services, ports, and workflow.

https://claude.ai/code/session_01NPLijSpjCg48htC37k2AkM
@intendednull intendednull merged commit 729f2af into main Mar 29, 2026
@intendednull intendednull deleted the claude/cli-deploy-services-6HOqy branch March 29, 2026 11:02
intendednull added a commit that referenced this pull request Apr 26, 2026
lifecycle, fix IrohBlobStore spec drift, track 4 new follow-ups

Round 2 review (two fresh agents) verified all 15 round-1 fixes
land cleanly with no regressions, then surfaced 8 new findings
(0 critical, 3 medium, 5 low) by widening scope to cross-component
interactions, perf, and API surface.

Fixed inline (trivial doc / spec):

- Add an "Actor coordination signal" row to the spec decision tree
  + CLAUDE.md table covering tokio::sync::watch / oneshot /
  broadcast / Notify, with the explicit rule that
  tokio::sync::Mutex is forbidden for business state on the same
  terms as std/parking_lot Mutex. Closes the spec gap that left
  contributors without guidance on async channels. (round-2 #3)
- Reconcile spec § 184 with the corrected IrohBlobStore comment
  (round-1 fixed the code, missed the spec). The blob store is
  not an iroh-callback boundary — it's an interim stub. The relay-
  status timestamp Mutex stays in the iroh boundary list. (round-2 #4)
- Document the web `_event_loop` drop pattern in `crates/web/src/app.rs`
  so future readers see explicitly that the actor System is process-
  scoped on web (page reload tears everything down) and that any
  actor needing pre-close cleanup must route via `beforeunload`,
  not Drop. (round-2 #8)

Tracked as new follow-ups in spec § Follow-up work:

- F5. SearchActor head-of-line + rebuild-storm fix. Rebuild blocks
  Query in FIFO order; the rebuild Effect has no debounce. Fix is
  chunked-Rebuild + Debounce<Rebuild> wrap. (round-2 #1, #2 — Med)
- F6. Browser-tier coverage for SearchIndexHandle consumers. The
  spawn_local + Effect path has no wasm-pack test. (round-2 #6)
- F7. Sealed ClientSpawner to narrow the system() API surface,
  rather than exposing the full SystemHandle. (round-2 #7)
- F8. Search-query debouncing-flicker fix via generation tag or
  Leptos Resource migration. (round-2 #5)

Each follow-up has a "Trigger:" line naming the dedicated PR title.

`just check` green: clippy zero warnings, 1003+ tests pass, WASM
compile clean. Loop terminates here per the user's two-round cap;
no Critical issues remain.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

2 participants