fix(container): gracefully handle missing on_wake column#2410
Merged
Conversation
…tion session DBs The container opens inbound.db read-only, so it can't ALTER TABLE. If the host hasn't run migrateMessagesInTable yet (e.g., container rebuilt before host restart), the on_wake column won't exist and the query crashes, causing a restart loop. Detect the column via PRAGMA table_info and conditionally include the filter clause. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
carstenf
pushed a commit
to carstenf/nanoclaw
that referenced
this pull request
May 11, 2026
Brings in 66 upstream commits including:
- fix(container): gracefully handle missing on_wake column in pre-
migration session DBs (nanocoai#2410)
- feat(db): container_configs table — per-group runtime config moves
from groups/<folder>/container.json into the central DB, materialized
to the file at spawn time. One-time backfill on startup migrates
existing container.json content.
- feat(cli-scope): scopeField on groups, sessions, destinations,
members; hardens ncl cross-scope operations.
- chore: rename remaining qwibitai/nanoclaw references → nanocoai/nanoclaw
- CI workflow guards + docs cleanup + version bumps to 2.0.56
Conflict resolution in src/group-init.ts: kept upstream's
`ensureContainerConfig(group.id)` + our local `ensureHindsightWired`
call. Rewrote src/group-init-hindsight.ts to use the DB layer
(getContainerConfig / updateContainerConfigJson) instead of the
file-based readContainerConfig / writeContainerConfig that upstream
removed.
Build + 341 host tests pass on the merged tree.
Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
zenmindhacker
pushed a commit
to zenmindhacker/nanoclaw
that referenced
this pull request
May 26, 2026
…egrade fix(container): gracefully handle missing on_wake column
tamasPetki
pushed a commit
to tamasPetki/nanoclaw
that referenced
this pull request
Jun 4, 2026
…egrade fix(container): gracefully handle missing on_wake column
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
inbound.dbread-only — it can'tALTER TABLEto add theon_wakecolumnmigrateMessagesInTable) after pulling the on-wake PR, the container query crashes on the missing column, causing a restart loopPRAGMA table_info(cached) and conditionally includes the filter clauseTest plan
on_wake filteringtests pass (28 tests)🤖 Generated with Claude Code