Skip to content

fix(cli): replace retired Basic Memory links#1083

Merged
phernandez merged 1 commit into
mainfrom
codex/1026-fix-doc-links
Jul 16, 2026
Merged

fix(cli): replace retired Basic Memory links#1083
phernandez merged 1 commit into
mainfrom
codex/1026-fix-doc-links

Conversation

@phernandez

Copy link
Copy Markdown
Member

Why

The link fixes proposed in #1026 are valid, but that external-contributor PR remains blocked on the CLA. This is an independent maintainer implementation; no contributor commit was copied.

The retired documentation paths and subscription URL return 404s. The CLI also retained the retired subscription URL as its fallback when the cloud API omits a subscribe_url.

What changed

  • update the README knowledge-format and CLI-reference links to their canonical documentation paths
  • update the cloud CLI guide to use the pricing page
  • update the CLI subscription-required fallback to use the pricing page
  • add a regression test for the missing-subscribe_url fallback

Server-provided subscription URLs still take precedence.

Validation

  • uv run pytest tests/cli/test_cloud_authentication.py -q — 10 passed
  • just fast-check
  • verified the three replacement URLs return HTTP 200

Risk

Low. The runtime change only affects the fallback URL when the cloud API does not provide one.

Supersedes #1026.

Signed-off-by: phernandez <paul@basicmachines.co>
@phernandez

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 1caa5c1f90

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@phernandez
phernandez merged commit ce17082 into main Jul 16, 2026
23 checks passed
@phernandez
phernandez deleted the codex/1026-fix-doc-links branch July 16, 2026 16:46
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