Document AidboxMigration execution-type and improve 2607 release notes - #38
Merged
Merged
Conversation
Adds documentation for running AidboxMigration SQL outside a transaction (HealthSamurai/sansara#8102), needed for statements PostgreSQL forbids inside a transaction block such as CREATE INDEX CONCURRENTLY. * migrations.md: new "Run SQL outside a transaction" section covering the execution-type parameter, which request contexts allow it (direct POST and batch bundles yes, transaction bundles rejected), a batch Init Bundle example, and the no-rollback caveat. * how-to-run-sql-via-init-bundle.md: use CREATE INDEX CONCURRENTLY as the main example, since a plain CREATE INDEX blocks writes for the duration of the build. * indexes/README.md: new "Applying an index" section listing the three routes for autocommit DDL. * sql-endpoints.md: cross-link from Aidbox-Sql-Autocommit to the migration route. * release-notes.md: rework the 2607 section with documentation links and fuller descriptions. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Aitem
approved these changes
Aug 6, 2026
* Zen seed: sansara c95e50057a removes the seed/seed-v2 engines and migrates default OAuth2/SMART scopes to auth module resources; f9bcd85aa0 drops the SeedImport resource type. * C-CDA: sansara 32be24501 (merged in 085fb373e) removes the CCDA module, closing sansara#6558. The converter now ships separately. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
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.
Documents running
AidboxMigrationSQL outside a transaction, added in HealthSamurai/sansara#8102, and reworks the 2607 release notes.Why
CREATE INDEX CONCURRENTLY(and other DDL PostgreSQL forbids inside a transaction block) previously could not run through any Aidbox migration path. The newexecution-typeparameter makes it possible, but nothing in the docs mentioned it.Changes
configuration/migrations.md— new "Run SQL outside a transaction" section: theexecution-typeparameter, a value table (in-transactiondefault vsnot-in-transaction), which request contexts allow it (direct POST to either API format andbatchbundles work; atransactionbundle is rejected with 422), abatchInit Bundle example, and the no-rollback caveat with the query for finding invalid indexes. Also adds a row to theAidboxMigrationvsPOST /db/migrationscomparison table.tutorials/other-tutorials/how-to-run-sql-via-init-bundle.md— switched the main example toCREATE INDEX CONCURRENTLY, since a plainCREATE INDEXlocks the table against writes for the duration of the build. That required the bundle type to becomebatch, theexecution-typeparameter, and a version bump to 2607. Added a step checkingindisvalid, and a note that a failedbatchentry warns rather than blocking startup.deployment-and-maintenance/indexes/README.md— new "Applying an index" section. The page listed which indexes to create but never how to apply DDL that cannot run in a transaction. Covers$psqlwithAidbox-Sql-Autocommit, SQL Console autocommit mode, andAidboxMigration.api/rest-api/other/sql-endpoints.md— cross-link from theAidbox-Sql-Autocommitheader row to the migration route.overview/release-notes.md— reworked the 2607 section: documentation links throughout, fuller descriptions for SMART Health Cards and batch validation, typo fixes, and formatting matched to the surrounding releases.Note
The sansara PR description documents the parameter as
runInTransaction: false(avalueBoolean), but the merged code implementsexecution-typeas avalueCodewith valuesin-transaction/not-in-transaction. These docs follow the code and tests inproto/migration.clj; the PR description looks stale.Verification
bun lintpasses: 11 checks, no new warnings. All new anchor links verified against their target headings.🤖 Generated with Claude Code