Skip to content

dev-app rule: a board schema bump needs a dev app rebuild - #959

Merged
m4ttheweric merged 1 commit into
mainfrom
dev-app-board-schema-rule
Oct 30, 2026
Merged

m4ttheweric merged 1 commit into
mainfrom
dev-app-board-schema-rule

Conversation

@m4ttheweric

@m4ttheweric m4ttheweric commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

The dev-app rule said served apps run from source, which is true only of the deck-served board. Cron triage and the status writers the board hands launched panes run the compiled Contents/Helpers/board, so after a board SCHEMA_VERSION bump that binary refuses the migrated state db until the dev app is rebuilt. Found right after #958 (state db v7).

  • Adds one bullet to .claude/rules/dev-app.md

Docs only. Proposing to skip the CodeRabbit wait: it's one paragraph.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Clarified which board components run from source and which use the bundled app.
    • Noted that rebuilding the dev app may be needed after a database schema change.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Walkthrough

The dev app instructions now distinguish the board served from source on port 11006 from board cron triggers and status writers that use the bundled helper. They document that a schema version bump can make the older helper refuse the migrated database until the dev app is rebuilt from the merged ref.

Changes: Dev app operational guidance

Layer / File(s) Summary
Clarify helper execution after schema updates
.claude/rules/dev-app.md
The instructions distinguish the source-served board from the bundled helper used by cron triggers and status writers. They describe the helper's behavior after a schema version bump and the rebuild required to use the merged ref.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Possibly related PRs

  • m4ttstack/mattstack#950: Raises the board state database schema from v5 to v6 and adds a migration.
  • m4ttstack/mattstack#958: Raises the board state database schema from v6 to v7 and adds the migration that prompted this documentation update.
Merge Risk: Minimal · up to 9cbbb

No merge-blocking risk was identified in this documentation-only change.

Architecture risk: Low · up to 9cbbb

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .claude/rules/dev-app.md: Added instructions distinguishing the source-served board from board cron triggers and status writers, which use the bundled helper. They specify that after a deploy bumps SCHEMA_VERSION, the older helper refuses the migrated state database until the dev app is rebuilt from the merged ref.
Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main documentation change: a board schema bump requires rebuilding the dev app.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@m4ttheweric
m4ttheweric merged commit 870166b into main Oct 30, 2026
19 checks passed
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