Skip to content

docs(feature-research): re-verify replit/windsurf upstream URLs and fix wrong topics#1651

Closed
Chen17-sq wants to merge 1 commit into
dyoshikawa:mainfrom
Chen17-sq:docs/1616-reverify-urls
Closed

docs(feature-research): re-verify replit/windsurf upstream URLs and fix wrong topics#1651
Chen17-sq wants to merge 1 commit into
dyoshikawa:mainfrom
Chen17-sq:docs/1616-reverify-urls

Conversation

@Chen17-sq
Copy link
Copy Markdown

Summary

Addresses item 2 from #1616 — the Collect-step-4 re-verification pass that the rulesync-feature-research SKILL.md now mandates for new map entries.

I ran a fresh fetch against the three suspect rows. Results:

File Row Old URL What the page actually is Action
replit.md mcp (L10) https://docs.replit.com/tutorials/agent-skills "How to use Agent Skills in your projects" — Skills, not MCP Downgrade to documented sentinel pair (No dedicated upstream MCP surface in map)
windsurf.md commands (L11) https://docs.windsurf.com/windsurf/cascade/memories "Memories & Rules" — not commands Swap to https://docs.windsurf.com/windsurf/cascade/workflows (H1 "Workflows") — confirmed to exist
windsurf.md skills (L13) https://docs.windsurf.com/windsurf/cascade/memories "Memories & Rules" — not skills Swap to https://docs.windsurf.com/windsurf/cascade/skills (H1 "Skills") — confirmed to exist

windsurf.md L8 rules is kept on the Memories URL: that page's H1 is literally "Memories & Rules", so it is the canonical doc for the rules row.

For the Codex CLI rows the issue called out as candidates (/codex/subagents, /codex/skills, /codex/hooks), all three pages exist with matching H1s — no change needed.

Test plan

  • pnpm cicheck (fmt + oxlint + eslint + tsgo + 5550 vitest + sync-skill-docs check + cspell + secretlint) passes locally on Node 22.
  • Pre-commit pnpm dev generate also runs cleanly — the generated mirrors under .claude/, .opencode/, .github/, .takt/ regenerate without complaint.
  • Tables are re-aligned by oxfmt to match the new URL widths.
  • No Upstream-surface text edits. The residual mixed-cell wording (e.g. ; no Rulesync-supported …, Rulesync maps …) is item-1 / item-4 territory and is intentionally out of scope here.

Refs #1616 (item 2 of 5 — items 1 and 3 were addressed in #1647 and #1646; items 4 and 5 are still open).

…ix wrong topics

Addresses item 2 in dyoshikawa#1616. The Collect-step-4 re-check pass surfaced three rows whose Official-docs URL did not actually describe the feature claimed by the row. WebFetch results:

- replit.md L10 mcp: https://docs.replit.com/tutorials/agent-skills resolves to the "Agent Skills" page (H1 "How to use Agent Skills in your projects"), not MCP. No dedicated Replit MCP doc page was found, so downgrade the row to the documented sentinel pair ("No dedicated upstream MCP surface in map" / "No Rulesync-supported Replit MCP target in map").

- windsurf.md L11 commands: https://docs.windsurf.com/windsurf/cascade/memories resolves to "Memories & Rules". A dedicated Workflows page does exist at https://docs.windsurf.com/windsurf/cascade/workflows (H1 "Workflows"); swap the URL.

- windsurf.md L13 skills: same Memories page is the wrong target. A dedicated Skills page exists at https://docs.windsurf.com/windsurf/cascade/skills (H1 "Skills"); swap the URL.

windsurf.md L8 rules keeps the Memories URL — it is the legitimate canonical page for both Memories and Rules per its H1.

Tables re-aligned by oxfmt to match the new URL widths. No Upstream-surface text edits; the residual mixed-cell wording is item-1 / item-4 territory and is intentionally out of scope here.
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your contribution! Unfortunately, you currently have 9 open PRs (including this one), which exceeds the limit of 2 for external contributors.

Please wait for an existing PR to be reviewed/merged, or close one before opening a new one. See CONTRIBUTING.md for details.

@Chen17-sq
Copy link
Copy Markdown
Author

Closing this proactively — the github-actions bot flagged that I'm well over the 2-PR limit for external contributors documented in CONTRIBUTING.md. I opened these nine PRs in one batch without spotting that rule first, which was my mistake — sorry for the noise.

The patch in this PR is ready as-is (pnpm cicheck green, lint-staged green, pre-commit pnpm dev generate green) and stays small enough for a quick read; reopening is one click.

Keeping #1643 (refactor(utils): consolidate isPlainObject, addresses #1638) and #1644 (refactor(copilot-rule): reuse toPosixPath, addresses #1603) as my current two. I'll reopen this one — addressing #1616 item 2 — once a slot frees up.

@Chen17-sq Chen17-sq closed this May 16, 2026
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