Skip to content

[AAASM-4061] 🐛 (python-sdk): Fix brew install command (wrong org)#210

Merged
Chisanan232 merged 2 commits into
masterfrom
v0.0.1/AAASM-4061/fix/homebrew_tap_command
Jul 4, 2026
Merged

[AAASM-4061] 🐛 (python-sdk): Fix brew install command (wrong org)#210
Chisanan232 merged 2 commits into
masterfrom
v0.0.1/AAASM-4061/fix/homebrew_tap_command

Conversation

@Chisanan232

Copy link
Copy Markdown
Contributor

Description

The user-facing "install manually" hint printed by the Python SDK when the aasm
runtime binary is missing pointed at a non-existent Homebrew tap. The command
brew install agent-assembly/tap/aasm is missing the ai- org prefix, so Homebrew
resolves it to github.com/agent-assembly/homebrew-tap — an org that does not exist —
and the install fails with a 404. The canonical command is
brew install ai-agent-assembly/tap/aasm.

This PR corrects the command in both places it appears in the SDK, and updates the
README repo-table row to point at the renamed homebrew-tap repository (previously
labelled and linked as homebrew-agent-assembly).

Type of Change

  • ✨ New feature
  • 🔧 Bug fix
  • ♻️ Refactoring
  • 🍀 Performance improvement
  • 📚 Documentation update
  • 🚀 Release

Breaking Changes

  • No
  • Yes (please describe below)

Related Issues

  • Parent bug: AAASM-4060
  • Subtask: AAASM-4061

Exact Changes

  • agent_assembly/runtime.py (INSTALL_HINT): brew install agent-assembly/tap/aasmbrew install ai-agent-assembly/tap/aasm
  • agent_assembly/_install.py (INSTALL_HINT): brew install agent-assembly/tap/aasmbrew install ai-agent-assembly/tap/aasm
  • README.md (repo table): link github.com/ai-agent-assembly/homebrew-agent-assemblygithub.com/ai-agent-assembly/homebrew-tap, and visible label homebrew-agent-assemblyhomebrew-tap

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No tests required (explain why)

These are user-facing string edits (install hints + a doc link); there is no logic
change to test. Validation performed:

  • python -m py_compile agent_assembly/runtime.py agent_assembly/_install.py → exit 0.
  • ruff is not installed on this machine, so the lint step was skipped (noted).
  • grep residual checks all return zero: no bare agent-assembly/tap without the
    ai- prefix, no ai-ai- doubling, no residual homebrew-agent-assembly.

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Comments added for complex logic
  • Documentation updated if needed
  • All tests passing

🤖 Generated with Claude Code

@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sonarqubecloud

sonarqubecloud Bot commented Jul 4, 2026

Copy link
Copy Markdown

@Chisanan232

Copy link
Copy Markdown
Contributor Author

✅ Claude Code review — AAASM-4061

1. CI: Green. All required checks pass (SonarCloud pass); skipping jobs are the release-only docs/e2e-codecov stages that don't run on a non-release PR.
2. Scope: Covers the subtask fully — runtime.py:41 + _install.py:34 install hints now ai-agent-assembly/tap/aasm; README.md:277 tap link/label → homebrew-tap. Grep: 0 residual bare agent-assembly/tap, 0 homebrew-agent-assembly.
3. Side effects: None. Edits are inside user-facing hint strings + a README table link; no code path, API, or business logic touched. The old agent-assembly/tap/aasm was a hard 404 (org doesn't exist) — this strictly fixes broken UX.
4. Front-end: N/A — no FE code in this PR.

Verdict: ready to approve + merge (mergeable; BLOCKED = awaiting required Pioneer approval). — Claude Code

@Chisanan232 Chisanan232 merged commit 5cf4643 into master Jul 4, 2026
26 checks passed
@Chisanan232 Chisanan232 deleted the v0.0.1/AAASM-4061/fix/homebrew_tap_command branch July 4, 2026 00:53
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