Commit Graph

590 Commits

Author SHA1 Message Date
Patrick szymkowiak 7dbef2ce00 fix(init): remove telemetry info lines from init output
Telemetry details are already documented in the README privacy section.
No need to print them during init.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
2026-04-03 13:37:52 +02:00
aesoft 5bd35a33ad Merge pull request #952 from rtk-ai/fix/pr-934-review
fix(review): PR #934
2026-03-31 20:45:44 +02:00
aesoft bae7930119 fix(review): PR #934 2026-03-31 20:43:59 +02:00
Florian BRUNIAUX 13e37bf7a8 ci: enforce test presence on new/modified filter modules
Add check-test-presence.sh script and CI job that fails if any *_cmd.rs
file added or modified in a PR is missing a #[cfg(test)] block.

- New CI job runs with no dependencies (parallel to all other jobs, <10s)
- Uses --diff-filter=AM to catch both added files and test deletions
- --self-test mode for local verification
- Add missing tests to wget_cmd.rs (17 tests) and env_cmd.rs (12 tests)
  covering pure functions: compact_url, format_size, parse_error,
  extract_filename, mask_value, is_lang_var, is_cloud_var, etc.

Fixes the enforcement gap: CONTRIBUTING.md required tests but CI did not
check. Now 34/34 *_cmd.rs modules have #[cfg(test)].

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
2026-03-31 12:02:53 +02:00
Florian BRUNIAUX 7e6452c543 Merge pull request #528 from ousamabenyounes/fix/rtk-disabled-warning
warn: emit stderr warning on RTK_DISABLED=1 (#508)
2026-03-31 11:37:24 +02:00
aesoft 14265b48c3 Merge pull request #933 from rtk-ai/maintainers-apply-doc
fix(docs): add maintainers docs
2026-03-30 19:56:32 +02:00
aesoft 0c3aad8156 Update MAINTAINERS_APPLY.md 2026-03-30 19:45:04 +02:00
aesoft 47a76ea35e Merge pull request #932 from rtk-ai/refacto-folders-and-documentation
fix(refacto-p2): more standardize
2026-03-30 19:37:45 +02:00
aesoft 2f7278ac59 fix(docs): add documentation 2026-03-30 19:36:27 +02:00
aesoft 5af18f87ae Merge branch 'develop' into refacto-folders-and-documentation 2026-03-30 19:29:34 +02:00
Adrien Eppling d75e864f20 fix(rules): add wc RtkRule with pattern field for develop compat
Aligns with PR #884 (wc rewrite support) already on develop.
2026-03-30 19:26:44 +02:00
aesoft 92c671a175 Merge pull request #910 from rtk-ai/refacto-files-and-constants
fix(refacto-p2):  more standardize
2026-03-30 18:51:37 +02:00
aesoft 75bd607d55 Merge pull request #904 from rtk-ai/refacto-folders-and-documentation
fix(refacto-p1): unified cmds execution flow  (+ rm dead code)
2026-03-30 18:43:54 +02:00
Florian BRUNIAUX bbe3da642b fix(ci): fix validate-docs.sh broken module count check
Two issues:
1. Module count compared top-level `mod` in main.rs (8) against
   "Total: 64 modules" in ARCHITECTURE.md -- incompatible metrics,
   always fails. Replaced with a simple .rs source file count (informational only).
2. CLAUDE.md check for Python/Go commands was too strict -- these
   commands belong in README.md (user-facing), not CLAUDE.md (Claude Code guidance).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
2026-03-30 18:26:04 +02:00
Florian BRUNIAUX a9def11139 chore(skills): add missing frontmatter to all project-level skills
All 8 .claude/skills/ files were missing effort, tags, and allowed-tools
fields. Also added name field to issue-triage, pr-triage, and rtk-triage
which had none. Effort levels inferred from content analysis.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
2026-03-30 18:22:34 +02:00
Adrien Eppling 0a8b8fd069 fix(review): address PR #910 review feedback
- container.rs: warn on kubectl JSON parse failure instead of silent fallback
- runner.rs: fix savings inflation when filter_stdout_only by tracking stdout only
- README.md: document automod behavior in module registration step
2026-03-30 18:13:39 +02:00
Adrien Eppling f3217a467b fix(registry): quoted env prefix + inline regex cleanup + routing docs
- ENV_PREFIX now handles FOO="bar baz", FOO='val', escaped quotes
- Move 6 inline lazy_static regex to module level
- Merge rewrite_head_numeric + rewrite_tail_lines → rewrite_line_range
- Consolidate head/tail call sites in rewrite_segment
- Add rewrite pipeline documentation to TECHNICAL.md
- Rewrite discover README as concept/flow documentation
- Add lexer-based tokenizer for shell command parsing

Co-Authored-By: Andrew Hundt <ATHundt@gmail.com>
2026-03-30 17:58:03 +02:00
aesoft 08aabb95c3 fix(standardize): merge pattern into rues 2026-03-30 17:58:02 +02:00
aesoft 7fd221f446 fix(standardize): git+kube sub wrappers run_filtered 2026-03-30 17:58:01 +02:00
aesoft 234909d2c7 fix(automod): add auto discovery for cmds 2026-03-30 17:58:00 +02:00
aesoft e48485adc6 fix(cmds): more covering for run_filtered 2026-03-30 17:57:59 +02:00
aesoft aabc0167bc fix(cleaning): constant extract 2026-03-30 17:57:57 +02:00
aesoft ba9fa345f3 fix(cmds): migrate remaining exit_code to exit_code_from_output 2026-03-30 17:55:15 +02:00
Adrien Eppling c1c56f24d7 style: fix cargo fmt in ls.rs assert macros
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-30 15:02:17 +02:00
aesoft 495a152059 fix(emots): replace 📊 with "Summary:" 2026-03-30 14:41:21 +02:00
Adrien Eppling 9521dcab83 Merge remote-tracking branch 'origin/master' into develop
# Conflicts:
#	src/hooks/init.rs
2026-03-30 12:45:18 +02:00
Florian BRUNIAUX c54afe3da8 Merge pull request #884 from manulqwerty/fix/hook-rewrite-wc 2026-03-30 10:22:55 +02:00
aesoft 48ec28d531 fix(go): align go tool golangci-lint with Result<i32> 2026-03-29 14:08:45 +02:00
aesoft 23c0b7e433 fix(cloud): reuse early exit for edge case 2026-03-29 13:40:57 +02:00
aesoft 078ad1d503 fix(cmds): add run_filtered wrapper to remaining cmd 2026-03-29 13:40:57 +02:00
aesoft 6878819798 fix: remove dead code 2026-03-29 13:40:56 +02:00
aesoft 32ff4663cd fix(cmds): exit code unified return code flow 2026-03-29 13:40:54 +02:00
aesoft 77a0951433 fix(cmds): shared cmd execution flow + clean & update docs
shared cmd exec flow and also removed some notes in technical docs (does not belong here)
2026-03-29 13:40:10 +02:00
aesoft 6444c4b018 Merge pull request #903 from rtk-ai/release-please--branches--master--components--rtk
chore(master): release 0.34.1
dev-0.35.0-rc.85 dev-0.35.0-rc.88 dev-0.35.0-rc.87 dev-0.35.0-rc.86 v0.34.1
2026-03-28 22:27:59 +01:00
github-actions[bot] ad5a155e45 chore(master): release 0.34.1 2026-03-28 21:11:03 +00:00
aesoft 92996b1272 Merge pull request #898 from rtk-ai/cherry-pick-security-fix
fix(telemetry): hash + salt
2026-03-28 22:10:35 +01:00
aesoft 5eae11d164 fix(security): set 0600 permissions on salt file 2026-03-28 14:14:13 +01:00
aesoft 22dc059310 fix(telemetry): cache salt in-process 2026-03-28 14:14:13 +01:00
aesoft 33195cc686 fix(telemetry): docs + real info from "rtk init -g" 2026-03-28 14:14:13 +01:00
aesoft 51f9c888b8 fix(security): missing toml pkg 2026-03-28 14:07:53 +01:00
aesoft 32fdbbbb69 fix(security): salt device hash for telemetry 2026-03-28 14:07:53 +01:00
aesoft 2d2f199983 Merge pull request #883 from rtk-ai/hotfix/telemetry-salt-hash
fix(security): salt device hash for telemetry
2026-03-28 14:00:26 +01:00
aesoft 8ab0f3da47 Merge pull request #706 from shwoop/go-tool-golangci
feat(go): Support golangci-lint via go tool
2026-03-28 11:47:57 +01:00
aesoft 8e42120ea3 fix(security): set 0600 permissions on salt file 2026-03-28 10:46:08 +01:00
Florian BRUNIAUX 9b9075cbb1 Merge pull request #847 from MauroDeryckere/fix/cat-flag-rewrite
Correct fix. cat flags like -A/-v/-e have no rtk read equivalent — skipping rewrite is the right behavior. cat -n correctly preserved. Well tested.
2026-03-28 02:07:23 +01:00
Florian BRUNIAUX a0d6874f7b Merge pull request #677 from TechWizard9999/fix/cargo-clippy-show-error-details
LGTM after code review. truncate() correctly imported, error_details block placed before rule frequency table, test covers the main case. Minor duplication for error[Exxxx] style but not a bug.
2026-03-28 01:57:32 +01:00
Florian BRUNIAUX 9ecdc02aee Merge pull request #753 from sveitser/fix-shebangs
Standard portability fix. /bin/bash is not guaranteed on NixOS or macOS with Homebrew bash. /usr/bin/env bash is the correct practice.
2026-03-28 01:53:02 +01:00
Florian BRUNIAUX 5a0144d185 Merge pull request #763 from mvanhorn/osc/759-include-finished-line-in-cargo-output
Correct. The Finished line carries useful signal (build time + profile). Defensive implementation with Option fallback.
2026-03-28 01:50:18 +01:00
Florian BRUNIAUX 0a5d5d5392 Merge pull request #650 from tobwen/fix/psql-disable-help-flag
Correct fix. disable_help_flag = true is the right pattern for passthrough commands — consistent with other commands in the codebase.
2026-03-28 01:49:32 +01:00
Florian BRUNIAUX da67c3e46a Merge pull request #562 from yogasw/fix/gain-history-utc-to-local-time
Correct fix. UTC timestamps in a local CLI history view is a UX bug — chrono::Local is already a dependency.
2026-03-28 01:48:40 +01:00