Skip to content

fix: harden Copilot binding cleanup and config recovery - #1066

Merged
sabbour merged 2 commits into
devfrom
feat/fre-lockout-screen
Aug 31, 2026
Merged

fix: harden Copilot binding cleanup and config recovery#1066
sabbour merged 2 commits into
devfrom
feat/fre-lockout-screen

Conversation

@sabbour

@sabbour sabbour commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • harden platform/project Copilot binding cleanup and token revocation safety
  • treat malformed BYOK or credential payloads as unconfigured instead of breaking admin recovery
  • migrate and validate platform-default Copilot bindings safely across SQLite/Postgres moves

Changeset exemption: Follow-up hardening for the already-documented platform-default Copilot feature shipped in #1063.

Testing

  • dotnet test tests/Agentweaver.Tests/Agentweaver.Tests.csproj --filter "FullyQualifiedName!~PostgresIntegration"
  • dotnet test tests/Agentweaver.Tests/Agentweaver.Tests.csproj --filter "FullyQualifiedNamePlatformDefaultCopilotBindingServiceTests|FullyQualifiedNameProjectCopilotBindingServiceTests|FullyQualifiedName~GitHubConnectionsCredentialVaultTests"
  • npm run build
  • npm run test -- App.test.tsx PlatformSettingsPage.test.tsx

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

Copilot-Session: 20ea1af6-c1fb-441c-8c7d-086f7900fc16
@sabbour
sabbour enabled auto-merge (squash) August 31, 2026 21:05
@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs sync reminder

This PR changes code in doc-relevant paths but does not touch docs/**.
If this change adds or alters a feature, API endpoint, MCP tool, blueprint, or workflow,
please update the docs in the same PR (the definition of done includes docs).

Changed source files
apps/Agentweaver.Api/Endpoints/AuthEndpoints.cs

How to update fast

  • Invoke the agentweaver-docs-sync skill (.copilot/skills/agentweaver-docs-sync/SKILL.md) for the playbook.
  • Regenerate auto-derived reference: node scripts/gen-docs.mjs then commit.
  • Build to verify: cd docs && npm ci && npm run build.

This is a non-blocking reminder — it will not prevent merge. See .github/DOCS_SYNC.md.

@sabbour sabbour added the changeset:not-required This PR intentionally has no user-facing release note; rationale required in the PR body label Aug 31, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 20ea1af6-c1fb-441c-8c7d-086f7900fc16
@sabbour
sabbour merged commit 7dd046c into dev Aug 31, 2026
17 checks passed
@sabbour
sabbour deleted the feat/fre-lockout-screen branch August 31, 2026 21:21
@sabbour sabbour mentioned this pull request Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changeset:not-required This PR intentionally has no user-facing release note; rationale required in the PR body

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant