refacto(cmds): strip decorator noise from filter output#2289
Merged
Conversation
Separator lines (═══, ---) and an emoji status marker cost tokens without adding signal for the LLM — RTK output must never add noise over raw. Semantic labels are kept; the emoji is swapped for plain monochrome unicode.
Open
thehoff
added a commit
to thehoff/contextcrawler
that referenced
this pull request
Jun 7, 2026
…port) Port upstream rtk-ai/rtk rtk-ai#2289 (16d6599): remove pure-decoration tokens from LLM-bound filter output. Output-string-only — no logic changes. RTK output should never add noise over the raw command; these decorators cost tokens with zero semantic value to an agent. Stripped across ~16 command filters: - Box-drawing separator lines (═══…) — cargo, go, golangci, lint, next, tsc, mypy, pip, pytest, ruff, prettier, prisma, format, dotnet, rspec, and the format_tool_failure helper in display_helpers. - Dash headers → plain labels: "--- Changes ---" → "Changes:" (git), "--- first 10 lines ---" → "first 10 lines:" (wget); dotnet dash rules dropped. - Emoji ❌ → monochrome ✗ (rspec, incl. test assertions). Deliberately KEPT (human-TTY / not filter output): the gain dashboard period-table and progress bar, and the tracking inflation-test fixtures. Ported manually — a cherry-pick of 16d6599 conflicted on fork divergence (cargo/rspec cap constants + tee logic) and would have dragged in unrelated logic; the decorator edits were applied by hand instead. No insta snapshots in this fork (inline asserts only); affected assertions updated. Savings only increase (less fixed overhead per invocation). council-reviewed (codex + mmax, file-by-file, "clean"). 3024 tests pass (7 pre-existing hooks::integrity /tmp-mode env failures unrelated). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
thehoff
added a commit
to thehoff/contextcrawler
that referenced
this pull request
Jun 7, 2026
… pivot Tag the current develop as the fork's first own release (contextcrawler-v0.2.0), bundling the 2026-06-06 fix batch + the first external community contribution (library build, PR #185 by Danny Wilson). This is the baseline we branch the "CLI consumes the library" pivot from. Bumps Cargo 0.1.10 -> 0.2.0 (fork's own version line; upstream rtk v0.x tags are unrelated, hence the fork-namespaced tag). CHANGELOG [0.2.0] documents: - SECURITY: rtk-ai#2286 never-auto-allow + the live-path Ask fix. - FIXED: tsc/mypy/next lying-success filters. - PERF: grep/find 40->67%, rtk-ai#2289 decorator strip. - ADDED: library MVP (output_summary + lib.rs), with the dead-code caveat that the bin does not yet consume the lib (next-release work). council-reviewed (metadata-only, approved). No source changes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Separator lines (
═══,---) and an emoji status marker added tokens to filtered output without giving the LLM any extra signal — the summary headers and section labels already delimit the content.RTK output must never add noise over the raw command. This strips those decorators across the command filters, keeps the semantic labels (e.g.
Changes:), and replaces the one emoji marker with plain monochrome unicode.No behavior change beyond cleaner output.
cargo fmt/clippyclean, all tests pass.