fix: point CLI setup/docs links at the real repo, not a stale fork username - #155
Merged
Conversation
…ername doctor.py and full_setup.py's QUICKSTART_URL, and the IDE-rules template generated by ide_rules.py, all linked to nhadaututtheky/surreal-memory — a different GitHub username than the actual repo (acidkill/surreal-memory, per pyproject.toml's own [project.urls] and `gh repo view`). Likely a leftover from a fork/rename that was never fully updated. `smem doctor`'s "full setup guide" link and the generated IDE rules file both pointed at someone else's fork.
The previous commit swapped the wrong repo owner (nhadaututtheky -> acidkill) but changed the URL shape from a GitHub Pages link to a github.com blob link, which broke test_full_setup.py::TestQuickstartUrl::test_url_is_valid (asserts "github.io" in QUICKSTART_URL) and ruff format (line over 100 chars). Keeps the original acidkill.github.io/surreal-memory/guides/quickstart/ shape, just with the corrected username.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
doctor.py/full_setup.py'sQUICKSTART_URLand the IDE-rules template generated byide_rules.pyall linked tonhadaututtheky/surreal-memory— a different GitHub username than the actual repo (acidkill/surreal-memory, perpyproject.toml's own[project.urls]andgh repo view --json nameWithOwner). Likely a leftover from a fork/rename that was never fully updated.smem doctor's "full setup guide" link now points at the real repo's quickstart doc.smem full-setup's equivalent link, same fix.ide_rules.py(written into agent-instruction files) now links to the real repo instead of someone else's fork.Chosen replacement URL:
pyproject.toml's own already-canonicalDocumentationURL (https://github.com/acidkill/surreal-memory/blob/main/docs/getting-started/quickstart.md) — verified to exist, rather than guessing at a GitHub Pages or custom-domain URL that may not be live (themkdocs.ymlsite_url,https://surrealmemory.theio.vn, does not currently resolve).Out of scope (found, not fixed here)
src/surreal_memory/server/static/dist/assets/index-*.js) embeds apolar.sh/nhadaututthekydonation link — that's a build artifact; the real fix belongs in the dashboard's frontend source, not a hand-edit of the compiled bundle.nhadaututthekyalso appears acrossREADME.md,AGENTS.md,SECURITY.md,CODE_OF_CONDUCT.md,CHANGELOG.md, and severaldocs/pages — out of scope for this narrowly-targeted CLI fix.Test plan
grep -rn nhadaututtheky src/**/*.py— zero remaining occurrences (excluding the unrelated compiled JS bundle noted above)pyproject.toml's own canonicalDocumentationlink