Skip to content

Fix nullable worktree inventory upserts - #1015

Merged
chubes4 merged 1 commit into
mainfrom
fix/1014-nullable-inventory-upsert
Aug 7, 2026
Merged

Fix nullable worktree inventory upserts#1015
chubes4 merged 1 commit into
mainfrom
fix/1014-nullable-inventory-upsert

Conversation

@chubes4

@chubes4 chubes4 commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

  • omit null normalized inventory values before database replace operations
  • advance the inventory schema to version 2 so active installs rerun the additive schema definition and receive lifecycle-intent columns
  • cover SQLite nullable upserts plus lifecycle-intent persistence after a version-1 table migration

Fixes #1014

Verification

  • php tests/sqlite-worktree-inventory-contention.php
  • php tests/worktree-add-lifecycle.php
  • homeboy review lint --changed-only --placement=local --summary

AI assistance

OpenAI gpt-5.6-sol via OpenCode implemented and verified the nullable inventory upsert and additive schema migration. Chris remains responsible for the change.

AI assistance: OpenAI gpt-5.6-sol via OpenCode implemented and verified the nullable inventory upsert and additive schema migration. Chris remains responsible for the change.
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.

Avoid nullable REPLACE translation failure in worktree inventory

1 participant