Skip to content

Fix import/console-tab bugs in a0 and add package run tests - #70

Merged
erinepshovel-code merged 1 commit into
mainfrom
claude/push-probe-tmp
May 25, 2026
Merged

erinepshovel-code merged 1 commit into
mainfrom
claude/push-probe-tmp

Conversation

@erinepshovel-code

Copy link
Copy Markdown
Collaborator
  • subagents.py hard-imported the optional claude_agent_sdk at module level, so importing a0.adapters crashed whenever the SDK was absent even though ClaudeAgentAdapter guards the same import. Make the import degrade gracefully with a stand-in AgentDefinition.
  • transcripts is a standalone page, not a metadata-driven console tab; its page-nav UI_META (path, no tab_id/sections) was aggregated by collect_ui_meta(), producing an unrenderable placeholder tab. Drop it from the console tab collection.
  • Fix stale import in test_aggregate_round_trips_through_cache_breakdown (cache_breakdown is a module function, not an attribute of a removed energy_registry instance).
  • Remove dead test_energy_registry_persistence.py: it targets an EnergyRegistry class / set_active_provider_persistent API that no longer exists (provider selection moved to the conduct slot).
  • Add tests/test_a0_package.py: imports every a0/ submodule, asserts the subagent registry loads without the optional SDK, and runs the a0.a0 CLI end-to-end.

https://claude.ai/code/session_018fWZmwRbdzw3WAofM8bARB

- subagents.py hard-imported the optional claude_agent_sdk at module
  level, so importing a0.adapters crashed whenever the SDK was absent
  even though ClaudeAgentAdapter guards the same import. Make the import
  degrade gracefully with a stand-in AgentDefinition.
- transcripts is a standalone page, not a metadata-driven console tab;
  its page-nav UI_META (path, no tab_id/sections) was aggregated by
  collect_ui_meta(), producing an unrenderable placeholder tab. Drop it
  from the console tab collection.
- Fix stale import in test_aggregate_round_trips_through_cache_breakdown
  (cache_breakdown is a module function, not an attribute of a removed
  energy_registry instance).
- Remove dead test_energy_registry_persistence.py: it targets an
  EnergyRegistry class / set_active_provider_persistent API that no
  longer exists (provider selection moved to the conduct slot).
- Add tests/test_a0_package.py: imports every a0/ submodule, asserts the
  subagent registry loads without the optional SDK, and runs the a0.a0
  CLI end-to-end.

https://claude.ai/code/session_018fWZmwRbdzw3WAofM8bARB
Copilot AI review requested due to automatic review settings May 25, 2026 09:13
@erinepshovel-code
erinepshovel-code merged commit 2e6db64 into main May 25, 2026
15 of 18 checks passed
@erinepshovel-code
erinepshovel-code removed the request for review from Copilot May 25, 2026 09:35
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.

2 participants