chore(repo): repository cleanup and engineering audit

[sreerevanth]

Summary by CodeRabbit

• New Features

  • Released AgentWatch version 0.2.0.
  • Added expanded product vision and engineering audit documentation.
  • Added new guides for extended setup, developer installation, and custom adapters.

• Documentation

  • Updated installation requirements to Python 3.12+ and the current package name.
  • Improved documentation navigation and replaced local links with portable references.

• Build & Reliability

  • Improved Docker build context handling and frontend compose configuration.
  • Enhanced dependency security scanning and cross-platform benchmark file handling.
  • Updated optional telemetry tests to skip gracefully when required support is unavailable.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
agent-watch	Ready	Preview, Comment	Jul 12, 2026 7:59am

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR updates repository build and CI configuration, aligns package and documentation version references, adds product vision and engineering audit documents, refreshes documentation navigation, and applies test-support and formatting cleanup.

Changes

Repository refresh

Layer / File(s)	Summary
Build and release alignment .dockerignore, .github/workflows/ci.yml, docker-compose.yml, pyproject.toml, benchmarks/*, agentwatch/cli/demo.py, scripts/owasp_test_harness.py	Build exclusions, security auditing, package metadata, UTF-8 file handling, demo version text, Docker configuration, and CLI formatting are updated.
Documentation and navigation updates README.md, docs/*, mkdocs.yml	Python installation guidance, package naming, internal links, document titles, navigation entries, and a resolved-issue note are updated.
Product vision and engineering audit VISION.md, issues.md	New product vision, architecture direction, scope, priorities, success metrics, and a repository engineering audit are documented.
Test support and formatting cleanup tests/*, how HEAD__main__.py	Test setup and optional telemetry skips are adjusted, while multiple test files receive formatting-only changes. Dockerfile.frontend is removed and frontend compose configuration points to Dockerfile.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• sreerevanth/AgentWatch#30: Updates installation documentation around the agentwatch-ai package.
• sreerevanth/AgentWatch#480: Relates to OpenTelemetry reasoning-trace tests and availability handling.
• sreerevanth/AgentWatch#571: Directly conflicts with the deletion of docs/issue-498-resolved.md.

Suggested labels: documentation, infra

Poem

A rabbit hops through builds so bright,
UTF-8 stars make files take flight.
Docs unfold and visions grow,
Tests skip what they don’t yet know.
“V0.2.0!” the burrow sings.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 14.81% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main repo cleanup and engineering-audit focus of the changeset.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch repo-cleanup

---

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.}

[github-actions]

🧪 PR Test Results

Check	Result
Tests (pytest tests/)	✅ success
Lint (ruff check .)	❌ failure
Coverage (agentwatch)	73.78%

_{Python 3.12 · commit 1eeeb2c}

[coderabbitai]

🧹 Nitpick comments (1)

tests/test_semantic_caching.py (1)

60-107: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Remove unnecessary @pytest.mark.asyncio decorators for consistency.

Three async tests in this file (test_semantic_cache_ttl at line 70, test_adapter_base_hooks at line 87, test_cache_hit_skips_provider_call at line 108) still carry @pytest.mark.asyncio, while the other three async tests (test_semantic_cache_exact_match, test_semantic_cache_fuzzy_match, test_semantic_cache_miss) do not. Since this PR is already reformatting these lines and the project configures asyncio_mode = 'auto', the decorators are redundant and should be removed for consistency.

As per coding guidelines: "Use asyncio_mode = 'auto' in pytest configuration — no need for @pytest.mark.asyncio on async tests".

♻️ Remove redundant decorators

 `@pytest.mark.asyncio`
 async def test_semantic_cache_ttl(monkeypatch):

 `@pytest.mark.asyncio`
 async def test_adapter_base_hooks(monkeypatch):

 `@pytest.mark.asyncio`
 async def test_cache_hit_skips_provider_call(monkeypatch):

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@tests/test_semantic_caching.py` around lines 60 - 107, Remove the redundant
`@pytest.mark.asyncio` decorators from test_semantic_cache_ttl,
test_adapter_base_hooks, and test_cache_hit_skips_provider_call in
tests/test_semantic_caching.py. Leave the async test definitions and their
behavior unchanged, relying on the project’s asyncio_mode = "auto"
configuration.

Source: Coding guidelines

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@tests/test_semantic_caching.py`:
- Around line 60-107: Remove the redundant `@pytest.mark.asyncio` decorators from
test_semantic_cache_ttl, test_adapter_base_hooks, and
test_cache_hit_skips_provider_call in tests/test_semantic_caching.py. Leave the
async test definitions and their behavior unchanged, relying on the project’s
asyncio_mode = "auto" configuration.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: afd7fead-d9aa-4f0b-8f84-dc931f04a481

📥 Commits

Reviewing files that changed from the base of the PR and between 902a288 and 1eeeb2c.

📒 Files selected for processing (33)

• .coverage
• .dockerignore
• .github/workflows/ci.yml
• .gitignore
• Dockerfile.frontend
• README.md
• VISION.md
• __main__.py
• agentwatch/cli/demo.py
• benchmarks/generate_cases.py
• benchmarks/run_eval.py
• docker-compose.yml
• docs/architecture.md
• docs/architecture_detailed.md
• docs/cli/getting-started.md
• docs/custom_adapters_tutorial.md
• docs/developer_setup.md
• docs/getting_started_extended.md
• docs/issue-498-resolved.md
• how HEAD__main__.py
• issues.md
• mkdocs.yml
• pyproject.toml
• scripts/owasp_test_harness.py
• tests/conftest.py
• tests/test_cli_share.py
• tests/test_db_failure.py
• tests/test_platform.py
• tests/test_router.py
• tests/test_semantic_caching.py
• tests/test_server.py
• tests/test_sync_safety.py
• tests/test_telemetry.py

💤 Files with no reviewable changes (4)

• docs/issue-498-resolved.md
• Dockerfile.frontend
• tests/conftest.py
• tests/test_db_failure.py