Legacy identifiers note, softened REST claim, llms.txt legacy line - #33
Merged
Conversation
- Get Started: 'Why 29next?' callout where a merchant first meets accounts.29next.com: former name, current hostnames, API headers and key namespace carry it, not scheduled to change. - About: 'core capabilities' instead of 'every capability' through REST. - llms.txt: the same legacy-identifiers line for agents. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
nextcommerce-docs | dc1c2b3 | Commit Preview URL Branch Preview URL |
Sep 03 2026, 09:15 AM |
next-devin
marked this pull request as ready for review
September 3, 2026 09:10
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Both prior issues on
No new issues were introduced in the changed lines. Files Reviewed (1 file in incremental diff)
Previous Review Summary (commit e419aca)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit e419aca)Status: 2 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (3 files)
Reviewed by minimax-m3 · Input: 25.2K · Output: 1.7K · Cached: 86.5K |
…me placeholder Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.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
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.
Follow-up to #32. Three wording changes that needed the product owner's call, now made.
What
accounts.29next.com: Next Commerce was formerly 29 Next; the account and store hostnames, API headers, and API key namespace still carry that name; they are current on every store and not scheduled to change. A clean agent evaluation on 2026-09-03 answered "is 29Next a different company?" only from a marketing-site banner, and a second run could not answer it at all from the docs sites./llms.txt: the same legacy-identifiers sentence in the header, so an agent that never reaches Get Started still sees it.Verified
npm run buildincluding the link audits, search index budget check passes. The callout and the About wording render inout/.Decision needed from — none; these were Alex's answers.
Sibling PR on developer-docs carries the developer-side changes (Spark as starter theme, Theme Kit links, disputes and campaign-order notes, reference-URL redirect).
🤖 Generated with Claude Code