Skip to content

fix: accept Claude URL elicitation notifications - #1383

Merged
avifenesh merged 1 commit into
mainfrom
automation/weekly-sweep-2026-08-17
Aug 16, 2026
Merged

fix: accept Claude URL elicitation notifications#1383
avifenesh merged 1 commit into
mainfrom
automation/weekly-sweep-2026-08-17

Conversation

@avifenesh

Copy link
Copy Markdown
Collaborator

Summary

  • accept the documented elicitation_url_dialog Notification matcher for CC-HK-025
  • refresh the five reviewed Claude Code specification baselines
  • update rule evidence, generated documentation, and changelog

Root cause

Claude Code added a distinct Notification matcher for MCP URL elicitations, but agnix's event-specific allowlist still rejected it.

Validation

  • env -u GLM_API_KEY UPDATE_BASELINES=true scripts/check-tool-releases.sh
  • node scripts/sync-rule-bookkeeping.js --check
  • cargo fmt --all -- --check
  • cargo test -p agnix-core notification_matchers -- --nocapture
  • cargo test --workspace
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cmp CLAUDE.md AGENTS.md

Closes #1381

@avifenesh
avifenesh marked this pull request as ready for review August 16, 2026 21:08

@revuto-review revuto-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an auto review done by revuto.


Revuto completed the review and found no evidence-backed concerns.

@avifenesh
avifenesh merged commit 29e0840 into main Aug 16, 2026
17 checks passed
@avifenesh
avifenesh deleted the automation/weekly-sweep-2026-08-17 branch August 16, 2026 22:08
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.

Spec Drift Detected - Review Required

1 participant