feat: add anteroom tool support (#60) - #61
Conversation
📊 Code Quality ReportLinting Issues |
Coverage report
Click to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||||||||||||||||||||||||||||||||
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #61 +/- ##
==========================================
+ Coverage 76.08% 76.19% +0.10%
==========================================
Files 83 84 +1
Lines 8158 8254 +96
==========================================
+ Hits 6207 6289 +82
- Misses 1951 1965 +14 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8558acbb29
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| target_path="ANTEROOM.md", | ||
| content=wrapped, | ||
| metadata={"section_based": True}, |
There was a problem hiding this comment.
By translating every instruction to the same ANTEROOM.md target, package installs with more than one instruction will currently drop later instructions under default ConflictResolution.SKIP: _install_instruction_component in devsync/cli/package_install.py checks target_file.exists() and skips instead of appending/replacing sections, so only the first translated instruction is installed. This makes multi-instruction package installs for Anteroom incomplete in normal usage.
Useful? React with 👍 / 👎.
1 participant
Summary
ANTEROOM.mdfile at the project root with HTML comment section markers (same pattern as Codex CLI and Amp)aroombinary or~/.anteroom/config directoryChanges
ai_tools
anteroom.py— new tool implementation with single-file section marker patterncapability_registry.py— register Anteroom with instruction + resource supportdetector.py— add Anteroom to tool detection and priority listtranslator.py— add AnteroomTranslator for package component translationcore
models.py— addANTEROOMtoAIToolTypeenum and valid IDE listcomponent_detector.py— addANTEROOM.mdto single instruction file detectiontests
test_ai_tools_anteroom.py— 20 unit tests covering all tool methodstest_ai_tools_detector.py— update tool count assertions (22 → 23)test_capability_registry.py— add Anteroom to registry and capability testsdocs
CLAUDE.md— add Anteroom to package structure, IDE capability, and component translation sectionsREADME.md— update IDE integration count (22 → 23)docs/index.md— update IDE integration count (22 → 23)Issue References
Closes #60
Test Plan
invoke test-unit(1625 tests, 1 pre-existing typer failure)invoke lintinvoke format --checkinvoke typecheckdevsync toolsdetects Anteroom when installeddevsync package installtranslates instructions to ANTEROOM.md formatVision Alignment
Supports IDE-agnostic and standards-first principles. Anteroom uses markdown files with HTML comment markers, consistent with existing single-file tools (Codex, Amp). No new dependencies, no new config options.