Skip to content

enh(tools): /status reports tool_filter state when enabled (#2028)#2051

Merged
bug-ops merged 1 commit intomainfrom
feat-2028-status-tool-filter
Mar 20, 2026
Merged

enh(tools): /status reports tool_filter state when enabled (#2028)#2051
bug-ops merged 1 commit intomainfrom
feat-2028-status-tool-filter

Conversation

@bug-ops
Copy link
Copy Markdown
Owner

@bug-ops bug-ops commented Mar 20, 2026

Summary

Add visibility into dynamic tool schema filter state in the /status command.

When [agent.tool_filter] is enabled, /status now displays:

Filter:    enabled (top_k=N, always_on=N, N embeddings)

When disabled (default), the line is omitted (zero noise).

Changes

  • crates/zeph-tools/src/schema_filter.rs: Add pub fn top_k() and pub fn always_on_count() accessor methods with unit test
  • crates/zeph-core/src/agent/mod.rs: Add conditional Filter line to handle_status_command() output
  • CHANGELOG.md: Document the enhancement

Testing

  • All 5528 unit/integration tests pass
  • Label alignment verified (11 chars, matching other /status fields)
  • Manual testing: filter line appears when enabled, silent when disabled

Validation

  • Architect ✓ — minimal 2-file design
  • Critic ✓ — alignment fix applied
  • Developer ✓ — implementation complete
  • Tester ✓ — test coverage adequate
  • Perf ✓ — zero overhead
  • Security ✓ — no risks
  • Impl-Critic ✓ — code quality solid
  • Code Reviewer ✓ — approved with CHANGELOG

Closes #2028

@github-actions github-actions bot added documentation Improvements or additions to documentation rust Rust code changes core zeph-core crate labels Mar 20, 2026
@bug-ops bug-ops enabled auto-merge (squash) March 20, 2026 14:43
@github-actions github-actions bot added the size/S Small PR (11-50 lines) label Mar 20, 2026
Add public accessor methods to ToolSchemaFilter (top_k, always_on_count) and
display filter configuration in /status output when enabled. Output shows:
  Filter:    enabled (top_k=N, always_on=N, N embeddings)

Label alignment matches existing 10-char column width. Silent when filter is
disabled (default) for zero noise.

- crates/zeph-tools/src/schema_filter.rs: Add accessors + test
- crates/zeph-core/src/agent/mod.rs: Add conditional Filter line to /status
- CHANGELOG.md: Document change
@bug-ops bug-ops force-pushed the feat-2028-status-tool-filter branch from bde5b9e to 57f3693 Compare March 20, 2026 14:45
@bug-ops bug-ops merged commit 58b835b into main Mar 20, 2026
25 checks passed
@bug-ops bug-ops deleted the feat-2028-status-tool-filter branch March 20, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core zeph-core crate documentation Improvements or additions to documentation rust Rust code changes size/S Small PR (11-50 lines)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

enh(tools): /status should report tool_filter state when enabled

1 participant