Skip to content

Add observer quarantine clear command, expose quarantine status, and fix dependency restore/build validation#16

Merged
einari merged 3 commits into
mainfrom
copilot/add-clear-quarantine-command
May 15, 2026
Merged

Add observer quarantine clear command, expose quarantine status, and fix dependency restore/build validation#16
einari merged 3 commits into
mainfrom
copilot/add-clear-quarantine-command

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 15, 2026

Summary

Adds observer quarantine operations in the CLI, exposes quarantine state in observer listings, and addresses dependency/CI validation issues found during PR feedback.

Added

  • Added chronicle observers clear-quarantine <OBSERVER_ID> with confirmation flow and clear success/error messaging.
  • Added runtime-compatible invocation for ClearObserverQuarantine so supported Chronicle contracts are used when available, with a validation error when unsupported.
  • Added specs for clear-quarantine invocation success and unsupported-contract fallback.
  • Added list-observer coverage to verify isQuarantined is present in JSON output.

Changed

  • Extended chronicle observers list output to include quarantine state:
    • JSON/compact JSON include isQuarantined.
    • table/plain output include a Quarantined column.
  • Updated package reference casing for MongoDB in integration tests to use MongoDB.Driver.
  • Updated Cli.Specs project warning configuration to avoid Release build failure from generated-source type conflicts (CS0436) in CI-equivalent builds.

Fixed

  • Fixed missing operator action in CLI for clearing observer-level quarantine.
  • Fixed observability gap where observers list did not explicitly indicate quarantine state.
  • Fixed CI restore/build failure caused by duplicate central package version entries for MongoDB (NU1506).
  • Verified CI-equivalent commands succeed: dotnet build --configuration Release and dotnet test --configuration Release.

Removed

  • Removed duplicate central package version entry that caused restore/build failures.

Security

  • No security-impacting behavior changes; security scan run as part of validation.

Deprecated

  • None.

Copilot AI linked an issue May 15, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add support for clearing quarantined observers Add observer quarantine clear command and expose quarantine status in observers list May 15, 2026
Copilot finished work on behalf of einari May 15, 2026 18:10
Copilot AI requested a review from einari May 15, 2026 18:10
@woksin woksin marked this pull request as ready for review May 15, 2026 19:57
@einari
Copy link
Copy Markdown
Contributor

einari commented May 15, 2026

@copilot update the Chronicle dependencies and verify that everything is working

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 15, 2026

@copilot update the Chronicle dependencies and verify that everything is working

Updated and verified. Chronicle package versions are already at the latest available (15.26.1), and I fixed the package-version duplication that was breaking CI restore/build. I also aligned the MongoDB package reference casing and verified with CI-equivalent commands: dotnet build --configuration Release and dotnet test --configuration Release (both passing). Commit: be2bcad.

Copilot AI changed the title Add observer quarantine clear command and expose quarantine status in observers list Add observer quarantine clear command, expose quarantine status, and fix dependency restore/build validation May 15, 2026
Copilot finished work on behalf of einari May 15, 2026 20:41
@einari einari added the minor New features, non-breaking additions label May 15, 2026
@einari einari merged commit bae78c6 into main May 15, 2026
1 check passed
@einari einari deleted the copilot/add-clear-quarantine-command branch May 15, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor New features, non-breaking additions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for clearing quarantine

2 participants