fix: detect async def test_ functions in project inspector

[Raghu :) (RogueTex)]

Description

The regex in src/navi_bootstrap/init.py (line 252) used to count test functions only matched synchronous def test_* functions:

# Before
count += len(re.findall(r"^\s*def test_", content, re.MULTILINE))

This missed async def test_* functions, causing test_count to be undercounted for projects using pytest-asyncio or any async test suite.

Type of Change

• Bug fix
• New feature
• Documentation
• Test coverage
• Refactor
• Security
• Breaking change

Checklist

• My code follows the project's style (ruff check src/navi_bootstrap/ tests/)
• I have added tests for my changes
• I have updated documentation if needed
• All existing tests pass (pytest tests/ -v)

Related Issues

Fixes #32

Additional Notes

The fix updates the regex to use a non-capturing optional group (?:async\s+)? so both sync and async test functions are detected:

# After
count += len(re.findall(r"^\s*(?:async\s+)?def test_", content, re.MULTILINE))

A new test test_detects_async_test_functions was added to TestDetectTestInfo to cover this case.

[Copilot]

Pull request overview

This PR fixes inspect_project’s test counting by updating detect_test_info() to detect both synchronous def test_* and asynchronous async def test_* pytest tests, and adds coverage for the async case.

Changes:

• Update the regex used to count tests to include optional async before def.
• Add a new test intended to verify async test functions are counted.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/navi_bootstrap/init.py	Expands the test-function counting regex to match async def test_* as well as def test_*.
tests/test_init.py	Adds a new test case for async test detection (currently introduced with invalid Python formatting).

[Raghu :) (RogueTex)]

Copilot open a new pull request to apply changes based on the comments in this thread

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

[Nelson Spence (Fieldnote-Echo)]

Hey Raghu :) (@RogueTex) — thanks for picking this up! The core fix is solid.

We're currently drafting contributor guidelines (CONTRIBUTING.md) to make expectations around formatting, local checks, and PR structure clearer. Once that's in place, we'll circle back with specific feedback on this PR.

Sit tight — more to come shortly.

[project-navi-bot]

All required CI checks passed. Auto-approved by navi-bot.

[project-navi-bot]

Auto-approved by navi-bot.

[Nelson Spence (Fieldnote-Echo)]

Hey Raghu :) (@RogueTex) — apologies for the mess here. We had some CI infrastructure changes going through (ruleset updates, check name standardization) and your PR got caught in the crossfire during the migration. We've reverted and recreated your changes in #42 with proper attribution.

We'll provide substantive feedback on the PR tomorrow. Thanks for your patience and your contribution!