Skip to content

fix(lint): remove unused imports in test_policy_store and test_simulator#6

Merged
DevanshuNEU merged 1 commit into
mainfrom
feat/cli-personality
Apr 15, 2026
Merged

fix(lint): remove unused imports in test_policy_store and test_simulator#6
DevanshuNEU merged 1 commit into
mainfrom
feat/cli-personality

Conversation

@DevanshuNEU

Copy link
Copy Markdown
Contributor

Made-with: Cursor

What does this PR do?

Why?

Type of change

  • Bug fix
  • New feature / command
  • Refactor (no behavior change)
  • Docs / chore
  • New extraction pattern (saar/extractors/)
  • Formatter change (saar/formatters/)

Module affected

  • saar/commands/ — CLI command logic
  • saar/extractors/ — Pattern extraction
  • saar/formatters/ — Output generation
  • saar/linter.py — AGENTS.md quality rules
  • saar/scorer.py — Quality scoring
  • saar/models.py⚠️ Core data contract (discuss before touching)

Verification

source venv/bin/activate

# 1. Lint
ruff check saar/ tests/

# 2. Tests — 548 must pass
pytest tests/ -q

# 3. Dogfood — saar must extract itself cleanly with no stale warnings
saar extract . --no-interview

# 4. If you changed a formatter, regenerate all context files
saar extract . --force --no-interview

Checklist

  • ruff check saar/ tests/ passes (no E/F violations)
  • pytest tests/ -q → 548 passed
  • saar extract . --no-interview runs clean (no stale fact warnings)
  • Did NOT modify saar/models.py without discussion
  • Did NOT add external dependencies (no Supabase, Redis, network calls in core path)
  • Version bumped in pyproject.toml + saar/__init__.py (if shipping a release)

@vercel

vercel Bot commented Apr 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
saar Ready Ready Preview, Comment Apr 15, 2026 1:34am

@DevanshuNEU DevanshuNEU merged commit 4bdf36d into main Apr 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant