fix(knowledge): preserve permission notices and fixture isolation

[waleedlatif1]

Summary

• Preserve permission warnings in the organization overview when sync notices are combined, matching the source detail page.
• Normalize Coda fixture identities before sharing mutations, and cover the guards and composed notices with regression tests.
• Run the organization overview PostgreSQL suite in CI and align its queued-sync expectations with the existing response contract.

Type of Change

• Bug fix

Testing

78 unit/UI tests and 13 real PostgreSQL tests pass. App type checking, lint, all 46 repository audits, and generated-artifact checks pass. The eight cleanup passes found no issues. Live Coda mutations were not rerun for this fixture-guard change.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Sep 17, 2026 8:06am UTC

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge; the latest changes make the PostgreSQL fixtures more deterministic without introducing a new correctness or rule-compliance issue.

Summary

This PR improves knowledge-connector operational reporting and live Coda fixture safety.

• Recognizes exact, line-delimited permission notices within composed synchronization diagnostics while avoiding substring false positives.
• Normalizes Coda fixture identities before allowing live sharing mutations.
• Adds regression coverage for permission-notice composition, paused connectors, fixture isolation, and queued synchronization states.
• Adds the organization overview PostgreSQL integration suite to CI.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Connector synchronization diagnostics] --> B{Paused?}
  B -->|Yes| C[Ignore operational permission notice]
  B -->|No| D[Split diagnostics into lines]
  D --> E{Exact permission notice present?}
  E -->|Yes| F[needs_attention: permission_sync_incomplete]
  E -->|No| G[Evaluate other synchronization failures]
  H[Coda live-sharing test] --> I[Fetch document name and owner]
  I --> J[Normalize owner and second identity]
  J --> K{Expected fixture name and distinct identities?}
  K -->|No| L[Refuse sharing mutation]
  K -->|Yes| M[Allow fixture mutation]

Loading

_{Reviews (3) · Last reviewed commit: "fix(knowledge): make healthy sync fixtur..."}

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

All reported issues were addressed across 7 files

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 7 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}