Skip to content

feat(cli): prototype recommendation output adapters - #113

Closed
sttts wants to merge 1 commit into
ai-dynamo:mainfrom
sttts:sttts-output-adapter-poc
Closed

sttts wants to merge 1 commit into
ai-dynamo:mainfrom
sttts:sttts-output-adapter-poc

Conversation

@sttts

@sttts sttts commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an explicit, repeatable --output NAME selector to aisimulate recommend
  • discover post-recommendation writers through a new aisimulate.output_adapters entry-point group
  • pass the same-named configuration section, final SweepResult, and prepared output directory to the selected writer
  • keep simulation selection (--stack) independent from artifact selection (--output)

This is an upstream PoC for ai-dynamo/dynamo#14282. AISimulate does not currently have an extensible output interface; this PR proposes the smallest final-result hook needed for Dynamo to provide --output dgd from the ai-dynamo package.

The PoC intentionally does not implement DGD rendering in AISimulate and does not add per-round callbacks. Canonical recommendation.json and recommendation YAMLs are written before optional output plugins run.

Validation

  • python/aisimulate/.venv/bin/pytest -c pytest.ini -p no:timeout tests/test_unified_cli.py tests/test_output_adapter_discovery.py tests/test_stack_discovery.py tests/sweeper/test_provider_discovery.py — 35 passed
  • python/aisimulate/.venv/bin/ruff check --config python/aisimulate/pyproject.toml python/aisimulate/src/aisimulate/main.py python/aisimulate/src/aisimulate/output_adapter.py tests/test_unified_cli.py tests/test_output_adapter_discovery.py
  • python/aisimulate/.venv/bin/python -m aisimulate.main recommend --help

Signed-off-by: Dr. Stefan Schimanski <stefan.schimanski@gmail.com>
@sttts

sttts commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor Author

Superseded by #334. #113 was accidentally closed during the rebase and cannot be reopened. Hence a new PR.

@sttts sttts closed this Sep 28, 2026

This branch was previously deployed

1 inactive deployment
pr-wheel-staging — 61aa3777 Deployed Sep 4, 2026 by copy-pr-bot[bot] via Application Wheel (arm64) #265
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