Skip to content

Comments

Fix tools CLI future import placement and remove duplicated shim#25

Merged
Victor-Dixon merged 4 commits intomainfrom
codex/fix-syntaxerror-in-cli.py
Jan 1, 2026
Merged

Fix tools CLI future import placement and remove duplicated shim#25
Victor-Dixon merged 4 commits intomainfrom
codex/fix-syntaxerror-in-cli.py

Conversation

@Victor-Dixon
Copy link
Owner

@Victor-Dixon Victor-Dixon commented Jan 1, 2026

Motivation

  • Fix a SyntaxError from from __future__ import annotations not being at the top of tools/cli.py which broke CI.
  • Enforce the SSOT intent that tools/cli.py is the single entry point for python tools/cli.py usage.
  • Remove duplicated shim content to avoid confusing or conflicting entry points.
  • Keep the file structure concise and under the project's Python LOC constraints.

Description

  • Removed the duplicated CLI shim block so from __future__ import annotations remains at the top of tools/cli.py.
  • Consolidated to a single main() implementation and the if __name__ == "__main__": guard invoking main().
  • Kept routing logic between the toolbelt and unified dispatcher intact via ToolRegistry, toolbelt_main, and unified_main.
  • No public API or runtime behavior changes beyond the import placement and cleanup.

Testing

  • No automated tests were run for this PR.
  • The repository CI previously failed with a SyntaxError caused by the misplaced future import, and this change addresses that specific error.

Codex Task


Note

Resolves a CI-breaking SyntaxError and simplifies the CLI entrypoint while preserving behavior.

  • Moves from __future__ import annotations to the top of tools/cli.py
  • Removes duplicated shim content and consolidates to a single main() and if __name__ == "__main__" guard
  • Retains routing logic between toolbelt and unified dispatcher via ToolRegistry, toolbelt_main, and unified_main

Written by Cursor Bugbot for commit c605444. Configure here.

@Victor-Dixon Victor-Dixon merged commit f389870 into main Jan 1, 2026
1 check passed
@Victor-Dixon Victor-Dixon deleted the codex/fix-syntaxerror-in-cli.py branch January 1, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant