Skip to content

Fix AMR install URL metadata#4006

Merged
Siri-Ray merged 1 commit into
nexu-io:mainfrom
TheAngryPit:codex/fix-amr-install-url
Jun 11, 2026
Merged

Fix AMR install URL metadata#4006
Siri-Ray merged 1 commit into
nexu-io:mainfrom
TheAngryPit:codex/fix-amr-install-url

Conversation

@TheAngryPit

Copy link
Copy Markdown
Contributor

Fixes #3989

Why

I am opening this as TheAngryPit agent after #3989 confirmed that the Open Design AMR install link points to the dead https://github.com/nexu-io/vela URL.

This is a user-facing broken install/help path. The repo already treats https://open-design.ai/amr as the canonical AMR page in README, landing header, and analytics metadata, and that page currently returns HTTP 200.

What users will see

When AMR is unavailable, the install action now points to https://open-design.ai/amr instead of a GitHub 404.

Surface area

  • UI — new page / dialog / panel / menu item / setting / empty state in apps/web or apps/desktop, including Electron menu bar
  • Keyboard shortcut — new or changed
  • CLI / env var — new od subcommand or flag, new tools-dev / tools-pack / tools-pr flag, or new OD_* env var
  • API / contract — new /api/* endpoint, new SSE event, or changed shape in packages/contracts
  • Extension point — new entry under skills, design-systems, design-templates, or craft, or change to the skills protocol
  • i18n keys — added new translation keys
  • New top-level dependency — adding any new entry to the root package.json
  • Default behavior change — changes what existing users experience without opting in
  • None — internal refactor, docs, tests, or translation update only

Screenshots

N/A. This changes the existing AMR install link target, not visible layout.

Bug fix verification

  • Test path that reproduces the bug: apps/daemon/tests/runtimes/env-and-detection.test.ts
  • Did the test go red on main and green on this branch? No. The added assertion directly covers the pre-fix metadata value from main, but the first local test attempt failed in setup because workspace links were stale after updating main. After refreshing workspace links with pnpm install, this branch is green.
  • Additional verification: https://open-design.ai/amr returns HTTP 200.

Validation

  • pnpm exec vitest run -c vitest.config.ts tests/runtimes/env-and-detection.test.ts from apps/daemon, 53 passed
  • pnpm --filter @open-design/daemon typecheck
  • pnpm guard, outside sandbox because tsx IPC cannot bind inside the sandbox here
  • curl -sSI https://open-design.ai/amr, HTTP 200

Agent involvement

Prepared by Aegis, working as TheAngryPit agent.

@lefarcen lefarcen added size/XS PR changes <20 lines risk/high High risk: apps/desktop, daemon, auth, migration, workflows, package deps type/bugfix Bug fix labels Jun 9, 2026
@lefarcen lefarcen requested a review from nettee June 9, 2026 16:04
@lefarcen

lefarcen commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Hey @TheAngryPit! 👋

The fix looks well-targeted — swapping the dead github.com/nexu-io/vela link for https://open-design.ai/amr is exactly what the investigation on #3989 identified, and the added installUrl assertion in the test suite locks it in properly.

Assigning @nettee for pool review. CI is already green, so you should hear back soon. ❤️

@nettee nettee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TheAngryPit I verified the changed ranges in apps/daemon/src/runtimes/metadata.ts and apps/daemon/tests/runtimes/env-and-detection.test.ts. The AMR unavailable-state install URL now points at the same canonical https://open-design.ai/amr destination already used elsewhere in the repo, and the added regression assertion covers the detectAgents() metadata path that surfaces this link to users. I also reran pnpm exec vitest run -c vitest.config.ts tests/runtimes/env-and-detection.test.ts in apps/daemon, and the targeted suite passed with 53/53 tests. Nice tight bugfix.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

@lefarcen lefarcen added needs-validation Runtime change detected; needs human or /explore agent validation. and removed needs-validation Runtime change detected; needs human or /explore agent validation. labels Jun 11, 2026
Merged via the queue into nexu-io:main with commit 913a69f Jun 11, 2026
15 checks passed
@kokisanai

Copy link
Copy Markdown
Contributor

Hi @TheAngryPit!

Your first Open Design PR has been merged! Huge thanks for jumping in and improving the project!

You contributed:

Merged PR: #4006 Fix AMR install URL metadata
#4006

Your first contribution improved the docs, which is one of the fastest ways to make Open Design easier for new contributors to use.

For your next contribution, we picked two issues that look like a good follow-up:

  1. wix-ja-slide / google-slides daemon: silent private-Drive fallback breaks image insertion under Workspace DLP #569 wix-ja-slide / google-slides daemon: silent private-Drive fallback breaks image insertion under Workspace DLP
    wix-ja-slide / google-slides daemon: silent private-Drive fallback breaks image insertion under Workspace DLP #569

This follows naturally from your first docs contribution and should be a clean next step in the same area.

  1. docs: comprehensive cloud deployment documentation #1021 docs: comprehensive cloud deployment documentation
    docs: comprehensive cloud deployment documentation #1021

This follows naturally from your first docs contribution and should be a clean next step in the same area.

If one of these looks interesting, feel free to comment /claim on the issue and we will help you get started!

Once your second PR gets merged, you will move into our Continuous Contributor tier. We are also starting to highlight repeat contributors more actively in the community, so this is a great time to keep going!

Thanks again for the first PR, and welcome to the Open Design contributor community!

The Open Design team

P.S. We hang out in Discord — come say hi: https://discord.gg/3C6EWXbdQQ
There's a #contributors channel where folks share what they're working on.

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

Labels

risk/high High risk: apps/desktop, daemon, auth, migration, workflows, package deps size/XS PR changes <20 lines type/bugfix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: AMR Install on 0.9.0 - going to dead link

4 participants