Skip to content

fix: Python 3.9 compatibility + s16 'list' command - #551

Open
songyang753-cloud wants to merge 2 commits into
shareAI-lab:mainfrom
songyang753-cloud:py39-compat-and-s16-list
Open

fix: Python 3.9 compatibility + s16 'list' command#551
songyang753-cloud wants to merge 2 commits into
shareAI-lab:mainfrom
songyang753-cloud:py39-compat-and-s16-list

Conversation

@songyang753-cloud

Copy link
Copy Markdown

Summary

  • fix(s09,s15): add from __future__ import annotations to make PEP 604 union annotations (str | None, etc.) import-safe on Python 3.9.
  • feat(s16): add a list command and register external plugin workflows with explicit stubs so they are discoverable via the runtime.

Verification

  • python3 s16_workflow_runtime/code.py list works on Python 3.9.6
  • load_integrated_host() + install_workflow_tool() no longer crash on import
  • review-changes demo runs end-to-end on Python 3.9

@songyang753-cloud

Copy link
Copy Markdown
Author

Local verification (Python 3.9.6)

All checks pass on the actual target interpreter. Note: the two red Vercel checks are account-level deployment blocks (Account is blocked) affecting preview deploys of fork PRs — unrelated to these changes, which are pure Python and don't touch the Vercel/Next.js site.

$ python3 --version
Python 3.9.6

$ python3 -c "import sys; sys.path.insert(0,'s09_memory'); import code"
s09 OK   # import-time union annotations no longer crash

$ python3 -c "import sys; sys.path.insert(0,'s15_integrated_harness'); import code"
s15 OK

$ python3 s16_workflow_runtime/code.py list
review-changes: ...
modernize-extract-rules: ...
modernize-harden-scan: ...
modernize-portfolio-assess: ...
modernize-reimagine-scaffold: ...
modernize-uplift-deltas: ...
modernize-uplift-migrate: ...

$ python3 -c "import sys; sys.path.insert(0,'s16_workflow_runtime'); import code as s16; s16.install_workflow_tool(s16.load_integrated_host())"
host loaded + tool installed OK

$ python3 s16_workflow_runtime/code.py demo
status=completed  agents=11  tokens=883
confirmed 6 real finding(s)

The previously-crashing load_integrated_host()load_memory_runtime()read_memory_file() path now imports and runs cleanly.

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