Skip to content

fix(cli): set up unconfigured workspaces on start - #1975

Merged
NicholaiVogel merged 1 commit into
mainfrom
hermes/hermes-4723ab34
Sep 29, 2026
Merged

NicholaiVogel merged 1 commit into
mainfrom
hermes/hermes-4723ab34

Conversation

@NicholaiVogel

Copy link
Copy Markdown
Collaborator

Summary

Unconfigured Signet workspaces now enter setup during daemon start, restart, or dashboard launch instead of failing before users can initialize them. Database-bearing workspaces missing manifests use recovery setup that preserves existing database contents. The curl installer also supports installing the latest nightly directly with --nightly.

Changes

  • Route fresh workspaces and eligible incomplete workspaces through interactive setup before daemon startup; retain the non-interactive signet setup guidance.
  • Repair workspaces that have a database but lack both manifests, using the workspace-layout-aware database path for legacy and v2 layouts.
  • Add --nightly to the curl installer; resolve the nightly version, select its release assets, and forward other installer arguments unchanged.
  • Document stable and nightly curl installation commands.

Type

  • fix — bug fix

Packages affected

  • @signet/cli / dashboard
  • @signet/web

Screenshots

Not applicable: this change modifies CLI setup and installer documentation, not a UI.

Testing

  • bun run typecheck passes
  • bun run lint passes
  • bun test passes
  • Tested against running daemon
  • N/A

Focused checks passed:

  • bun test surfaces/cli/src/features/setup-recovery.test.ts surfaces/cli/src/features/daemon.test.ts platform/core/src/workspace.test.ts scripts/install-shell-compat.test.ts — 55 passed.
  • bun run --filter '@signet/cli' build
  • bunx biome check README.md dist/signetai/README.md scripts/install-shell-compat.test.ts surfaces/cli/src/cli.ts surfaces/cli/src/features/daemon.ts surfaces/cli/src/features/daemon.test.ts surfaces/cli/src/features/setup-migrate.ts surfaces/cli/src/features/setup.ts surfaces/cli/src/features/setup-recovery.test.ts
  • bash -n web/marketing/public/install.sh and zsh -n web/marketing/public/install.sh
  • git diff --check

bun test surfaces/cli/src/features/setup.test.ts reports 43 passed and 2 failed on both this branch and origin/main: one existing onboarding test expects a dynamically allocated server port but receives the default port, and the related unavailable-workspace case times out. These failures reproduce on the unchanged base.

AI disclosure

  • AI tools were used (see Assisted-by tags in commits)

Notes

bun scripts/doc-drift.ts exits 1 on both origin/main and this branch because the API reference lists 35 routes not found by the current source inventory; this change does not modify API routes.

Assisted-by: Hermes-Agent:gpt-6-luna
@NicholaiVogel
NicholaiVogel merged commit dfcd133 into main Sep 29, 2026
23 of 24 checks passed
@NicholaiVogel
NicholaiVogel deleted the hermes/hermes-4723ab34 branch September 29, 2026 00:36
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