Fix YAML syntax errors in rust.yml workflow

[Copilot]

• Analyze GitHub Actions workflows and identify errors
• Fix YAML syntax errors in rust.yml workflow (brackets spacing, indentation)
• Test build and ensure project compiles successfully
• Verify workflow files pass YAML linting
• Fix failing tldr_tests in src/bin/cratedocs.rs

Summary

YAML Workflow Fixes

Fixed YAML syntax errors in .github/workflows/rust.yml:

• Removed extra spaces in brackets for branch definitions (lines 5, 7)
• Fixed indentation for steps (lines 17-21) to match GitHub Actions convention
• Verified project builds successfully with cargo check

Code Bug Fix

Fixed logic bug in src/tools/tldr.rs that caused test failures:

• Issue: When encountering consecutive LICENSE/VERSION headings, the function would skip the first but include the second in output
• Root cause: The condition if !skip && tldr_section_re.is_match(line) only started skipping if not already skipping, so a second LICENSE/VERSION heading would reset skip state and be included
• Fix: Changed to if tldr_section_re.is_match(line) to always skip LICENSE/VERSION headings regardless of current skip state

All tests now pass ✅

Original prompt

identify and fix the errors in the github actions

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[elasticdotventures]

@copilot when running the ci - it crashes because:
Running /home/runner/work/rust-cargo-docs-rag-mcp/rust-cargo-docs-rag-mcp/target/debug/deps/cratedocs-c774855e1e36c1b4

running 5 tests
test tldr_tests::test_apply_tldr_no_headings ... ok
test tldr_tests::test_apply_tldr_handles_no_license_or_versions ... ok
test tldr_tests::test_apply_tldr_malformed_markdown ... FAILED
test tldr_tests::test_apply_tldr_large_input ... FAILED
test tldr_tests::test_apply_tldr_removes_license_and_versions ... FAILED

failures:

---- tldr_tests::test_apply_tldr_malformed_markdown stdout ----

thread 'tldr_tests::test_apply_tldr_malformed_markdown' (8275) panicked at src/bin/cratedocs.rs:547:5:
assertion failed: !output.to_lowercase().contains("version")
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

---- tldr_tests::test_apply_tldr_large_input stdout ----

thread 'tldr_tests::test_apply_tldr_large_input' (8274) panicked at src/bin/cratedocs.rs:573:5:
assertion failed: !output.to_lowercase().contains("license")

---- tldr_tests::test_apply_tldr_removes_license_and_versions stdout ----

thread 'tldr_tests::test_apply_tldr_removes_license_and_versions' (8277) panicked at src/bin/cratedocs.rs:507:9:
assertion failed: !output.to_lowercase().contains("license")

failures:
tldr_tests::test_apply_tldr_large_input
tldr_tests::test_apply_tldr_malformed_markdown
tldr_tests::test_apply_tldr_removes_license_and_versions

test result: FAILED. 2 passed; 3 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.01s

error: test failed, to rerun pass --bin cratedocs
Error: Process completed with exit code 101.

[Copilot]

@copilot when running the ci - it crashes because:
Running /home/runner/work/rust-cargo-docs-rag-mcp/rust-cargo-docs-rag-mcp/target/debug/deps/cratedocs-c774855e1e36c1b4
...

Fixed in commit 6e79819. The issue was in the apply_tldr function logic - it wasn't properly handling consecutive LICENSE/VERSION sections. Changed the condition from if !skip && tldr_section_re.is_match(line) to if tldr_section_re.is_match(line) to ensure all LICENSE/VERSION headings are skipped regardless of current state. All tests now pass.