refactor: make CSV export default, consolidate instruction files

[nerdCopter]

• Remove --csv flag (CSV export always enabled for CLI binary)
• Update help text to clarify CSV is default output format
• Delete outdated .github/instructions/rust-project.instructions.md
• AGENTS.md is now single source of truth for project instructions

BREAKING CHANGE: --csv flag removed, CSV export is now always enabled

Summary by CodeRabbit

• New Features

  • CSV export is now always enabled by default; the --csv flag has been removed.

• Documentation

  • CLI help updated: clarified output directory applies to all outputs and notes default CSV with optional GPX/JSON.
  • Post-processing/help messages simplified to reflect always-on CSV export.

• Chores

  • Removed an internal project guidance document (no functional impact).

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR deletes a GitHub instructions markdown file and updates the CLI in src/main.rs to make CSV export always enabled, removes the --csv flag, generalizes output directory help text, and simplifies control flow by assuming CSV export is always performed.

Changes

Cohort / File(s)	Summary of Changes
Repo instructions cleanup .github/instructions/rust-project.instructions.md	Removed file containing Copilot instructions and internal guidance.
CLI defaults and output handling src/main.rs	Removed --csv flag; CSV export is now unconditional. Updated CLI descriptions to indicate default CSV export with optional GPX/JSON. Generalized output directory help text. Set ExportOptions.csv = true unconditionally and removed branches/messages dependent on the CSV flag.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly summarizes the two primary changes—making CSV export default and consolidating instruction files—directly reflecting the PR’s main refactors without unnecessary detail.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 20251008_fixup

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b1d3dbc and f9d5fe6.

📒 Files selected for processing (1)

• src/main.rs (3 hunks)

🧰 Additional context used

📓 Path-based instructions (4)

src/main.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Maintain the Rust application entry point at src/main.rs

Files:

• src/main.rs

**/*.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

**/*.rs: Do not remove or modify comments unless the related Rust code is changed; only add comments that explain code functionality (no AI instructional comments)
Never embed or call external binaries from Rust code

Files:

• src/main.rs

src/**/*.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Only commit Rust source files under src when committing code changes (no blanket git add)

Files:

• src/main.rs

{src,tests,benches,examples}/**/*.rs

📄 CodeRabbit inference engine (.github/instructions/rust-project.instructions.md)

{src,tests,benches,examples}/**/*.rs: Do not remove or modify comments unless the related Rust code is changed
Only add comments that explain code functionality; do not add AI instructional comments
Never embed or call external binaries from Rust code

Files:

• src/main.rs

🧠 Learnings (1)

📓 Common learnings

Learnt from: nerdCopter
PR: nerdCopter/bbl_parser#2
File: LICENSE_COMMERCIAL:1-4
Timestamp: 2025-08-29T19:52:05.099Z
Learning: nerdCopter prefers to avoid publishing personal information in license files for privacy and security reasons, as they are an individual maintainer rather than a company.

🔇 Additional comments (3)

src/main.rs (3)

264-264: Clear and accurate CLI description.

The updated about text clearly communicates that CSV export is the default behavior with optional GPX/JSON exports. This accurately reflects the breaking change and sets appropriate user expectations.

---

281-281: Good generalization of help text.

The updated help text appropriately reflects that the output directory can contain multiple file formats (CSV, GPX, event files), not just CSV. This is more accurate and flexible.

---

331-331: Well-implemented breaking change with clear documentation.

The hardcoded csv: true correctly implements the always-on CSV export behavior described in the PR. The inline comment clearly explains the rationale, which aids maintainability. This change ensures consistent CSV output while keeping GPX and event exports optional.

Memory usage is already managed by the existing safety limits at line 2107, so storing all frames for CSV export should not cause issues even with large log files.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3c6ba5f and b1d3dbc.

📒 Files selected for processing (2)

• .github/instructions/rust-project.instructions.md (0 hunks)
• src/main.rs (4 hunks)

💤 Files with no reviewable changes (1)

• .github/instructions/rust-project.instructions.md

🧰 Additional context used

📓 Path-based instructions (4)

src/main.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Maintain the Rust application entry point at src/main.rs

Files:

• src/main.rs

**/*.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

**/*.rs: Do not remove or modify comments unless the related Rust code is changed; only add comments that explain code functionality (no AI instructional comments)
Never embed or call external binaries from Rust code

Files:

• src/main.rs

src/**/*.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Only commit Rust source files under src when committing code changes (no blanket git add)

Files:

• src/main.rs

{src,tests,benches,examples}/**/*.rs

📄 CodeRabbit inference engine (.github/instructions/rust-project.instructions.md)

{src,tests,benches,examples}/**/*.rs: Do not remove or modify comments unless the related Rust code is changed
Only add comments that explain code functionality; do not add AI instructional comments
Never embed or call external binaries from Rust code

Files:

• src/main.rs

🧠 Learnings (1)

📓 Common learnings

Learnt from: nerdCopter
PR: nerdCopter/bbl_parser#2
File: LICENSE_COMMERCIAL:1-4
Timestamp: 2025-08-29T19:52:05.099Z
Learning: nerdCopter prefers to avoid publishing personal information in license files for privacy and security reasons, as they are an individual maintainer rather than a company.

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Test (windows-latest)
• GitHub Check: Test (macos-latest)

🔇 Additional comments (3)

src/main.rs (3)

264-264: LGTM! Clear documentation of the new default behavior.

The updated command description accurately reflects that CSV export is now the default format, with GPX and JSON as optional additions.

---

281-281: LGTM! Appropriately generalized help text.

The help text now correctly describes the directory as being for all output files (CSV, GPX, JSON) rather than just CSV files.

---

330-336: LGTM! CSV export now unconditionally enabled.

The hardcoded csv: true correctly implements the breaking change described in the PR. The retention of the ExportOptions.csv field (rather than removing it entirely) suggests this codebase may be used as a library where the field could still be useful for API consumers.