Skip to content

rust(chore): agent skill refresh and sift-cli 0.5.0 release prep - #763

Open
evan-sift wants to merge 5 commits into
rust/bulk-annotationsfrom
rust/mcp-skill-ga-refresh
Open

rust(chore): agent skill refresh and sift-cli 0.5.0 release prep#763
evan-sift wants to merge 5 commits into
rust/bulk-annotationsfrom
rust/mcp-skill-ga-refresh

Conversation

@evan-sift

@evan-sift evan-sift commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Description

Replaces #742, which GitHub's stack lock prevented from being retargeted after the stack restructure.

Two changes that close out the MCP tool expansion:

Refreshes the bundled Sift agent skill for the tools introduced below this branch in the stack. The skill's tool inventory now mirrors the server registry exactly (49 tools), and the workflow guidance covers calculated channel management, saved calculated channels through get_data, preview_rule dry runs, user-defined function discovery and versioning, and bulk update_annotation semantics (required annotation_ids list, per-id failure reporting).

Preps the sift-cli 0.5.0 release: version bump in sift_cli/Cargo.toml and a CHANGELOG.md entry covering the new tool surface, in the same format as the 0.4.4 entry.

Verification

  • cargo test -p sift_cli cmd::agent: 38 passed, 0 failed, including the skill/reference-bundle map test.
  • Inventory cross-check: every tool named in the skill exists in the server registry and tool_events.json (49 keys); the only names in the skill that are not registry tools are deliberate "there is no such tool" statements.
  • At this branch (the stack tip): cargo test -p sift_mcp 428 passed, cargo test -p sift_cli 218 passed, cargo fmt --all -- --check clean, cargo build -p sift_mcp --no-default-features clean.

carries `unresolved_calculated_channels`, the file is missing those columns:
tell the user which channels did not resolve rather than reporting the file as
complete.
- **Author or change a calculated channel.** A calculated channel is a SEL

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's CEL expression not SEL right?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think after this one is changed, looks good will approve 👍

Suggested change
- **Author or change a calculated channel.** A calculated channel is a SEL
- **Author or change a calculated channel.** A calculated channel is a CEL

Comment thread rust/crates/sift_cli/assets/skills/sift/SKILL.md
Comment thread rust/crates/sift_cli/assets/skills/sift/SKILL.md
@evan-sift
evan-sift force-pushed the rust/bulk-annotations branch from bc03eb9 to f0d6aee Compare August 27, 2026 04:40
@evan-sift
evan-sift force-pushed the rust/mcp-skill-ga-refresh branch from c71746d to 2b5451a Compare August 27, 2026 04:40
@evan-sift
evan-sift requested a review from lineville August 27, 2026 04:41
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.

2 participants