Skip to content

fix(python): make currency CLI entry point importable - #2829

Closed
tryambak2019 wants to merge 1 commit into
kagent-dev:mainfrom
tryambak2019:fix/2689-currency-cli-entrypoint
Closed

tryambak2019 wants to merge 1 commit into
kagent-dev:mainfrom
tryambak2019:fix/2689-currency-cli-entrypoint

Conversation

@tryambak2019

@tryambak2019 tryambak2019 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Restore the published currency package entry point and its container command.

Implemented

  • Replace the invalid absolute import with from .agent import graph.
  • Run the container through the installed currency console entry point.
  • Extract build_app() so main() and the test use the same ASGI construction path.
  • Add CI coverage that calls main() with Uvicorn mocked and verifies GET /health.

Validation

  • Local Python suite: 552 passed, 1 skipped.
  • uv lock --check and Ruff checks passed.
  • Currency image builds successfully.

#2689 Series

Related to #2689. This PR does not close it.

@tryambak2019
tryambak2019 requested a review from a team as a code owner September 15, 2026 21:32
@github-actions github-actions Bot added the bug Something isn't working label Sep 15, 2026
@tryambak2019
tryambak2019 force-pushed the fix/2689-currency-cli-entrypoint branch from 9b71114 to 8bebcdb Compare September 15, 2026 22:08
@github-actions github-actions Bot added bug Something isn't working and removed bug Something isn't working labels Sep 15, 2026
@tryambak2019

Copy link
Copy Markdown
Contributor Author

Superseded by #2887, which consolidates the requested Python startup and documentation changes for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant