Reframe docs repo for public AI-agent reference clones#11
Merged
Conversation
Clarify that the public repo exists for local agent reference, not outside contributions. Add AGENTS.md and LICENSE, team-only maintenance docs, CI, changelog URL repairs, and remove the internal readiness audit artifact. Co-authored-by: Cursor <cursoragent@cursor.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
nextcommerce-docs | eee033c | Commit Preview URL Branch Preview URL |
Jun 15 2026, 09:26 AM |
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| } | ||
|
|
||
| parsed.pathname = pathname; | ||
| return parsed.toString().replace(/\/$/, '') + (parsed.hash || ''); |
There was a problem hiding this comment.
CRITICAL: Duplicated hash bug — parsed.toString() already includes the hash, so appending parsed.hash || '' produces URLs with the hash written twice (e.g. .../3ds2-payments#3ds2-merchant-profiles#3ds2-merchant-profiles).
This is the root cause of the broken 3DS link in content/changelog/2023-09-12-admin-api-payments-campaigns-fulfillment.mdx:24. Re-running this script will keep emitting doubled fragments for any URL with a hash.
Suggested fix:
Suggested change
| return parsed.toString().replace(/\/$/, '') + (parsed.hash || ''); | |
| return parsed.toString().replace(/\/$/, ''); |
| - [Fulfillment Locations](https://docs.Next Commerce.com/start-here/get-started-on-29-next/fulfillment-settings#fulfillment-locations) settings list now shows a `Returns` column to easily see which fulfillment locations are configured to accept returns. | ||
| - [[3DS](https://docs.nextcommerce.com/docs/features/payments/3ds2-payments) Merchant Profiles](https://docs.Next Commerce.com/features/payments/3ds2-payments#3ds2-merchant-profiles) now have a `Name` field to use for reference to improve clarity for merchants with multiple 3DS merchant profiles with the same Merchant Name. | ||
| - [Fulfillment Locations](https://docs.nextcommerce.com/start-here/get-started-on-29-next/fulfillment-settings#fulfillment-locations) settings list now shows a `Returns` column to easily see which fulfillment locations are configured to accept returns. | ||
| - [3DSMerchant Profiles](https://docs.nextcommerce.com/docs/features/payments/3ds2-payments#3ds2-merchant-profiles#3ds2-merchant-profiles) now have a `Name` field to use for reference to improve clarity for merchants with multiple 3DS merchant profiles with the same Merchant Name. |
There was a problem hiding this comment.
CRITICAL: Two defects produced by the URL-fixing script:
- Link label is missing a space:
3DSMerchantshould be3DS Merchant(also visible elsewhere in the same sentence where the surrounding prose says "multiple 3DS merchant profiles"). - The URL has the hash fragment duplicated:
…#3ds2-merchant-profiles#3ds2-merchant-profiles. Browsers will treat the second#3ds2-merchant-profilesas a literal fragment id and fail to find it.
Both stem from fixDocsUrl in scripts/fix-changelog-urls.mjs:74 appending parsed.hash after parsed.toString() (which already includes the hash). Fix the script and re-run it; this entry will be regenerated correctly.
Suggested change
| - [3DSMerchant Profiles](https://docs.nextcommerce.com/docs/features/payments/3ds2-payments#3ds2-merchant-profiles#3ds2-merchant-profiles) now have a `Name` field to use for reference to improve clarity for merchants with multiple 3DS merchant profiles with the same Merchant Name. | |
| - [3DS Merchant Profiles](https://docs.nextcommerce.com/docs/features/payments/3ds2-payments#3ds2-merchant-profiles) now have a `Name` field to use for reference to improve clarity for merchants with multiple 3DS merchant profiles with the same Merchant Name. |
| ## Improvements | ||
|
|
||
| - [[3PL Central](https://docs.nextcommerce.com/docs/apps/3pl-central) App](https://docs.Next Commerce.com/apps/3pl-central) now features logs of API requests to assist merchants in troubleshooting order fulfillments when sent to their integrated fulfillment locations. | ||
| - [3PL CentralApp](https://docs.nextcommerce.com/docs/apps/3pl-central) now features logs of API requests to assist merchants in troubleshooting order fulfillments when sent to their integrated fulfillment locations. |
There was a problem hiding this comment.
WARNING: Link label has a missing space — 3PL CentralApp should read 3PL Central App. The original (pre-PR) line was [[3PL Central](…) App](…); the fix-up produced 3PL CentralApp because the inner text 3PL Central and the trailing word App were concatenated during the double-link collapse without inserting a separator.
Suggested change
| - [3PL CentralApp](https://docs.nextcommerce.com/docs/apps/3pl-central) now features logs of API requests to assist merchants in troubleshooting order fulfillments when sent to their integrated fulfillment locations. | |
| - [3PL Central App](https://docs.nextcommerce.com/docs/apps/3pl-central) now features logs of API requests to assist merchants in troubleshooting order fulfillments when sent to their integrated fulfillment locations. |
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
The incremental diff (
The only outstanding defect is the same sentence duplicated in the front-matter Issue Details (click to expand)No new issues on diff lines. Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments:
Files Reviewed (1 file in incremental diff)
Fix these issues in Kilo Cloud Previous Review Summaries (3 snapshots, latest commit 1099ebe)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 1099ebe)Status: 1 Issue Found | Recommendation: Address before merge Overview
Both issues from the previous review have been addressed in the incremental diff (
One new issue surfaced on the same line that was just edited. Issue Details (click to expand)WARNING
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments (carried forward from prior review):
Files Reviewed (2 files in incremental diff)
Fix these issues in Kilo Cloud Previous review (commit 5714998)Status: 2 Issues Found | Recommendation: Address before merge Overview
The previous review's 3 issues (duplicated Issue Details (click to expand)WARNING
SUGGESTION
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments:
Files Reviewed (11 files in incremental diff)
Previous review (commit 15113fe)Status: 3 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)CRITICAL
WARNING
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments:
Files Reviewed (39 files)
Reviewed by minimax-m3 · 189,124 tokens |
Remove duplicated hash fragments, preserve spaces when collapsing nested double-links, and normalize docs paths missing the /docs/ prefix. Co-authored-by: Cursor <cursoragent@cursor.com>
…orders.mdx Co-authored-by: kilo-code-bot[bot] <240665456+kilo-code-bot[bot]@users.noreply.github.com>
…kout.mdx Co-authored-by: kilo-code-bot[bot] <240665456+kilo-code-bot[bot]@users.noreply.github.com>
…kout.mdx Co-authored-by: kilo-code-bot[bot] <240665456+kilo-code-bot[bot]@users.noreply.github.com>
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
AGENTS.mdas the agent entry point (content/docs/vs developer-docs vs what to ignore)LICENSEwith local-reference usage termsdocs/MAINTENANCE.mdfor internal publishing workflow; removePUBLICATION_READINESS.md(audit complete)content/changelog.README.mddocs.Next Commerce.com, double-link artifacts, legacy API paths) across 28+ entries viascripts/fix-changelog-urls.mjsscripts/changelog-migration/as deprecatednpm ci,npm audit --omit=dev,npm run build)Test plan
npm run buildpassesnpm audit --omit=devcleanMade with Cursor