Skip to content

fix(migrations): use injected Postgres configuration - #1000

Merged
sabbour merged 1 commit into
devfrom
sabbour/repair-release-migration-factory
Aug 29, 2026
Merged

fix(migrations): use injected Postgres configuration#1000
sabbour merged 1 commit into
devfrom
sabbour/repair-release-migration-factory

Conversation

@sabbour

@sabbour sabbour commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • load PostgreSQL design-time migration configuration from app settings, injected environment variables, and Development user secrets without logging connection strings
  • forward -- --postgres-migrations from API, worker, and reference migration job bundle commands
  • preserve SQLite as the explicit default local design-time workflow and add regression coverage

Fixes #954

Validation

  • dotnet restore tests\Agentweaver.Tests\Agentweaver.Tests.csproj --locked-mode -p:CopilotSkipCliDownload=true
  • dotnet build tests\Agentweaver.Tests\Agentweaver.Tests.csproj --no-restore -p:CopilotSkipCliDownload=true
  • dotnet test tests\Agentweaver.Tests\Agentweaver.Tests.csproj --no-build --no-restore -p:CopilotSkipCliDownload=true --filter "FullyQualifiedName~MemoryDbContextDesignFactoryTests|FullyQualifiedName~PostgresMigrationManifestTests" (7 passed)
  • npm run version:check && npm run changeset:check

npm run docs:build could not run locally because the configured package-feed proxy returned 404 for mermaid@11.17.1 after npm run deps:ensure; no dependency files were changed.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3f94bc67-d29d-4d47-a8f7-5d99540c4b42
@sabbour
sabbour force-pushed the sabbour/repair-release-migration-factory branch from 6dda543 to 79c4ab0 Compare August 29, 2026 05:55
@sabbour
sabbour merged commit 1531361 into dev Aug 29, 2026
18 checks passed
@sabbour
sabbour deleted the sabbour/repair-release-migration-factory branch August 29, 2026 06:02
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