[w] wshm · Automated triage by AI

📊 Automated PR Analysis

Summary

Adds comprehensive Swift ecosystem support with 5 new TOML filters (swift test, swift package, swift run, swiftlint, swiftformat) and corresponding rewrite rules. Also includes minor pre-existing formatting and clippy fixes in unrelated files.

Review Checklist

• Tests present
• Breaking change
• Docs updated

Linked issues: #848

Analyzed automatically by wshm · This is an automated analysis, not a human review.

Hey

We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes src/ from a flat layout into subfolders.

No logic changes — only file moves and import path updates.

What you need to do

Rebase your branch on develop when receiving this comment:

git fetch origin && git rebase origin/develop

Git detects renames automatically. If you get import conflicts, update the paths:

use crate::git;        // now: use crate::cmds::git::git;
use crate::tracking;   // now: use crate::core::tracking;
use crate::config;     // now: use crate::core::config;
use crate::init;       // now: use crate::hooks::init;
use crate::gain;       // now: use crate::analytics::gain;

Need help rebasing? Tag @aeppling

@pszymkowiak

Need rebase after the folder re-organisation

BUG 1: swift-package.toml: match_output never fires

^Resolved uses a start-of-string anchor but the engine runs match_output before strip_lines. The blob starts with "Fetching...", not "Resolved". Regex is compiled without the multiline flag (Regex::new() at toml_filter.rs:349). Result: users get 3 lines of Computed/Resolved output instead of "ok (dependencies resolved)".

BUG 2: swiftlint.toml: violations summary stripped

^Done linting! strip pattern matches all "Done linting!" lines — including "Done linting! Found 3 violations, 1 serious". The violation count summary is lost. Test expects the summary in output but it's actually stripped.