Cleanup: format, lint, prune comments, fix tests, add CI - #113
Merged
Conversation
derekwisong
force-pushed
the
cleanup/black-and-comments
branch
3 times, most recently
from
May 14, 2026 02:14
dbd8847 to
539e626
Compare
- Add black + ruff as dev deps; format pyagent/ and tests/. - Strip narrative comments from pyagent/ source. - Prune stale planning docs; tighten the survivors; add an architecture page with mermaid diagrams and a runnable examples/quickstart.py. - Fix doc_tools sub-LLM result key (real bug); pin tree-sitter-language-pack to the 0.x line (Go grammar regression on 1.x); gate fastembed recall tests behind PYAGENT_HEAVY_TESTS. - Update stale stubs in 5 smoke tests so the suite is green again. - GitHub Actions CI: lint + smoke tests on push to main and PRs.
derekwisong
force-pushed
the
cleanup/black-and-comments
branch
from
May 14, 2026 02:21
539e626 to
135ec11
Compare
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pyagent/andtests/.pyagent/(~11% line reduction with no behavior change).docs/loses ~56% of its lines); adddocs/architecture.mdwith mermaid diagrams and a runnableexamples/quickstart.py.doc_tools._run_subllmwas reading the wrong key on LLM responses (text→content), soextract_doc/summarize_docwere broken against every provider.tree-sitter-language-pack <1.0— the 1.x line moved most grammars (including Go) to download-on-demand and hung offline.PYAGENT_HEAVY_TESTS=1so CI doesn't try to download a ~130MB model.black --check,ruff, and every smoke test on push tomainand on PRs.Diff: 123 files, +2,561 / −6,523 (~4,000 lines removed at no feature cost).
Test plan
pip install -e '.[dev]' && python -m black --check pyagent tests && python -m ruff check pyagent tests.for f in tests/smoke_*.py; do python -m tests.\$(basename \"\$f\" .py) || echo FAIL; done— expect every test to pass.README.md,docs/architecture.md,docs/plugin-design.md,docs/library-usage.md.examples/quickstart.pyruns end-to-end with a real API key set.