Conversation
The previous --schema flag name implied output type rather than behavior. --keys-only is explicit: values are shown by default, this flag strips them. Closes #621 (json values now preserved by default; --keys-only is opt-in). Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
Adds a one-liner listing all supported ecosystems (ruff, pytest, pip, golangci-lint, etc.) in the Architecture section to satisfy the pre-push documentation validation script. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
…rics Add 6 new fields to the anonymous daily telemetry ping to help identify which commands need filters and which filters need improvement: - passthrough_top: top 5 commands with 0% savings (missing filters) - parse_failures_24h: count of parse failures (filter fragility) - low_savings_commands: commands averaging <30% savings (weak filters) - avg_savings_per_command: unweighted average savings - hook_type: which AI agent hook is installed (claude/gemini/codex/etc) - custom_toml_filters: count of user-defined TOML filter files New tracking.rs queries: top_passthrough(), parse_failures_since(), low_savings_commands(), avg_savings_per_command(). Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Extend the daily anonymous ping with product-piloting metrics: Retention: first_seen_days, active_days_30d, commands_total Ecosystem: ecosystem_mix (category distribution percentages) Economics: tokens_saved_30d, estimated_savings_usd_30d Config: has_config_toml, exclude_commands_count, projects_count Features: meta_usage (gain, discover, proxy, verify, learn counts) Update README.md privacy section with full field-by-field table explaining what is collected and why it helps improve RTK. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Comprehensive telemetry documentation covering: - Why we collect (roadmap prioritization, filter quality, value measurement) - How it works (daily ping, background thread, fire-and-forget) - Every field with example values and purpose - What is NOT collected (explicit exclusion list) - Opt-out instructions - Data handling and privacy guarantees - Contributor guide for adding new fields Link added from README.md privacy section. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Creates docs/README.md (interface contract defining 3-tab structure, required frontmatter, and conventions) and 4 stub pages with valid frontmatter for docs/guide/, docs/reference/, and docs/architecture/. These are the prerequisites for the prepare-docs.mjs pipeline (Plan B) and for Adrien's Phase 1 content work. No existing files modified. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
Creates the complete user-facing guide for the docs website: Getting started: - quick-start.md: 5-minute walkthrough (init, first commands, rtk gain) - supported-agents.md: Claude Code, Cursor, Copilot, Gemini, Cline, Windsurf, Codex, OpenCode — integration tiers, install commands, graceful degradation Commands (adapted from FEATURES.md, English, --help-first format): - git.md: status/log/diff/show/add/commit/push/pull/branch + gh CLI - cargo.md: test/nextest/build/check/clippy/install + generic test/err wrappers - files.md: ls/read/grep/find/diff/wc/smart with before/after examples - javascript.md: vitest/playwright/tsc/eslint/prettier/next/pnpm/npm/npx/prisma - python.md: pytest/ruff/mypy/pip/deps Reference: - filters/using-filters.md: 8-stage pipeline, lookup priority, TOML DSL reference, Mermaid diagram (adapted from docs/filter-workflow.md) - analytics/gain.md: rtk gain flags, daily/weekly/monthly breakdowns, export formats, token estimation, database management (from docs/AUDIT_GUIDE.md) - configuration.md: full config.toml reference, env vars, tee system, telemetry - troubleshooting.md: common issues and fixes (from docs/TROUBLESHOOTING.md) All pages carry valid frontmatter (title, description, sidebar.order). No existing files modified. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
…mmands Guide — 10 new command and analytics pages: - commands/go.md: go test (NDJSON), golangci-lint - commands/ruby.md: rspec, rubocop, rake - commands/dotnet.md: dotnet build/test, binlog, format - commands/containers.md: docker ps/images/logs/compose, kubectl pods/services/logs - commands/github-cli.md: gh pr/issue/run, Graphite gt commands - commands/data.md: json, env, log, curl, wget, aws, psql, summary - commands/utilities.md: proxy passthrough, global flags, RTK_DISABLED - filters/creating-filters.md: TOML DSL guide, field reference, inline tests - analytics/discover.md: rtk discover — missed savings analysis - analytics/economics.md: rtk cc-economics — dollar savings vs ccusage Reference tab (9 pages): - contributing/guide.md: design philosophy, PR process, TOML vs Rust decision - contributing/security.md: vulnerability reporting, dangerous patterns, dep criteria - contributing/coding-standards.md: Rust rules, error handling, lazy_static, fallback pattern - contributing/testing.md: snapshot tests, token accuracy, cross-platform, benchmarks - internals/command-routing.md: 6-phase lifecycle, exit codes, verbosity, module map - internals/filter-pipeline.md: 12 filtering strategies, TOML stages, savings by ecosystem - internals/tracking-system.md: SQLite schema, data flow, token estimation, Tracker API - internals/hook-engine.md: rewrite registry, compound commands, exit code contract, rtk init - toml-dsl/specification.md: complete field reference, pipeline order, build compilation Architecture tab (4 pages): - diagrams/command-flow.md: end-to-end Mermaid — hook → RTK → LLM - diagrams/filter-pipeline.md: build pipeline + 8-stage runtime Mermaid - decisions/why-no-async.md: ADR — single-threaded, <10ms startup constraint - decisions/proxy-architecture.md: ADR — why CLI proxy over aliases/LD_PRELOAD/hooks-only All pages carry valid frontmatter (title, description, sidebar.order). No existing files modified. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
…flat docs - Remove docs/reference/ (10 files): duplicates CONTRIBUTING.md, SECURITY.md, ARCHITECTURE.md, src/*/README.md (distributed pattern from PR #869) - Remove docs/architecture/ (5 files): duplicates ARCHITECTURE.md ADRs and diagrams - Remove docs/guide/commands/ (12 files): showed rtk <cmd> syntax users never type (hooks rewrite transparently); content was derived from FEATURES.md - Remove docs/guide/filters/ (2 files): contributor content, belongs with src/filters/ - Remove docs/guide/analytics/discover.md, economics.md: implementation details - Remove legacy flat docs/ (6 files): FEATURES.md (FR), TECHNICAL.md, TROUBLESHOOTING.md, AUDIT_GUIDE.md, tracking.md, filter-workflow.md — all superseded by guide/ rewrites or codebase distributed docs - Add docs/guide/what-rtk-covers.md: replaces 12 command pages with one user-facing overview of 60+ commands by ecosystem, no rtk <cmd> syntax - Enrich docs/guide/analytics/gain.md: absorb advanced workflows from AUDIT_GUIDE.md (CI integration, pandas analysis, cron snapshots) - Adapt docs/guide/getting-started/quick-start.md: remove rtk <cmd> examples, explain hook transparency instead - Update docs/README.md: single-tab contract, pointer to codebase for technical docs - Update docs/guide/index.md: clean navigation matching final structure - Add Mermaid diagrams to src/filters/README.md: build pipeline + filter lookup (preserves content from filter-workflow.md in its natural home) Result: 9 files in docs/guide/ (was 39), zero legacy flat docs, zero duplicates. Codebase distributed docs (ARCHITECTURE.md, CONTRIBUTING.md, src/*/README.md) remain the source of truth for technical and contributor content. Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
Structure: - Move docs/README.md → .github/docs-pipeline-contract.md (interface contract is not user-facing content) - Rewrite docs/guide/index.md as a proper landing page with RTK intro, flow diagram, and navigation (replaces empty nav stub) - Move configuration.md → getting-started/configuration.md (sidebar order 4, after supported-agents per Adrien's suggestion) Missing content: - Add docs/guide/analytics/discover.md: rtk discover (missed savings analysis) and rtk session (adoption tracking across sessions) - Add uninstall instructions to installation.md (rtk init -g --uninstall, cargo uninstall, brew uninstall) - Add global flags section to what-rtk-covers.md (-u/--ultra-compact, -v/--verbose) - Add rtk smart command to what-rtk-covers.md Files section Agents: - Update supported-agents.md: 10 agents (was 9) - Add OpenClaw (TypeScript plugin, before_tool_call) - Add Mistral Vibe (planned, issue #800) - Fix GitHub Copilot: VS Code = transparent rewrite via PreToolUse, Copilot CLI = deny-with-suggestion (upstream limitation) Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
fix(ls): filter quality wrong truncation
pszymkowiak
added
bug
Collaborator
📊 Automated PR Analysis
SummaryFixes parsing of Review Checklist
Linked issues: #948 Analyzed automatically by wshm · This is an automated analysis, not a human review. |
- Fix broken links: docs/TROUBLESHOOTING.md and docs/AUDIT_GUIDE.md were removed in previous commit; update nav header and Documentation section - Condense Supported AI Tools: keep the summary table, replace 12 per-agent detail blocks with a single link to rtk-ai.app/guide/supported-agents - Condense Configuration: keep essential snippet and tee recovery example, link to rtk-ai.app/guide/configuration for full reference - Update Documentation section: lead with rtk-ai.app/guide as primary entry, keep INSTALL.md, ARCHITECTURE.md, CONTRIBUTING.md, SECURITY.md Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
…mary
When go test times out or is killed by a signal, the JSON stream
contains a package-level {"action":"fail"} with no Test field and no
FailedBuild field. The parser only handled test-level and build-level
failures, so this event was silently dropped. The summary then saw
0 passes + 0 fails and reported "No tests found" instead of the
actual failure.
The fix adds handling for package-level failures: a new else branch
in the fail handler, package-level output capture in the output
handler, and inclusion in the summary count and display.
Before: "Go test: No tests found"
After: "Go test: 0 passed, 1 failed in 1 packages" with the
timeout output shown under the package name.
Fixes #958
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…e, pricing P0 fixes: - top_passthrough: GROUP BY first word only, not full command args (privacy leak) - build_meta_usage: use dedicated count query instead of scanning deduplicated top list (was always returning 0 or 1) P1 fixes: - Single Tracker connection per ping (was opening 2 SQLite connections) - first_seen_days: propagate DB errors instead of unwrap_or(None) - ecosystem_mix: bound to 90 days instead of full table scan - avg_savings_per_command: compute per-command mean, not per-row mean - estimated_savings_usd_30d: use $3/Mtok (Sonnet input rate) not $5/Mtok Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app> Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
- passthrough_top example: "npm ci:8" -> "npm:8" (first word only) - estimated_savings_usd_30d: $5/Mtok -> $3/Mtok (input pricing) - Remove needless borrow in build_meta_usage (clippy) Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app> Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
docs: consolidate to user-facing guide only (removes duplicates and legacy flat docs)
Add rtk init --agent kilocode and rtk init --agent antigravity commands. Kilo Code: installs .kilocode/rules/rtk-rules.md (project-scoped) Google Antigravity: installs .agents/rules/antigravity-rtk-rules.md (project-scoped) Both follow the same prompt-level guidance pattern as Cline and Windsurf, using rules files that instruct the agent to prefix shell commands with rtk.
feat(telemetry): enrich daily ping with gap detection and quality metrics
fix: report package-level failures (timeouts, signals) in go test summary
feat(init): add support for kilocode and antigravity agents
fix(json): rename --schema to --keys-only, closes #621
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
6 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.
next release building
Summary
Bug fix / issue resolved
ISSUE #948 resolved by PR #990