Skip to content

Update Scaffold Docs - #2708

Open
zachfedor wants to merge 2 commits into
stellar:mainfrom
zachfedor:scaffold-stellar-tools-page
Open

zachfedor wants to merge 2 commits into
stellar:mainfrom
zachfedor:scaffold-stellar-tools-page

Conversation

@zachfedor

@zachfedor zachfedor commented Jul 28, 2026 •

Copy link
Copy Markdown
Contributor

Updates documentation following rename to "Stellar Scaffold" along with new GitHub organization and documentation website.

Also reduces some duplicated quickstart/tutorial info that is spread around the docs site. This became difficult to maintain in multiple places after any change to Scaffold's API. Instead, this adds pointers to our official documentation site which is deployed in lock-step with code changes ensuring there's a lot less outdated info.

NOTE: I did not change the file name since I didn't know how you all want to handle redirects. I can update that too, but the diff will be easier to read for now.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Renames Scaffold documentation to “Stellar Scaffold” and redirects detailed guidance to its synchronized external documentation.

Changes:

  • Updates naming, links, and navigation.
  • Condenses duplicated setup instructions.
  • Refreshes toolkit and registry descriptions.

Recommendation: NEEDS-CHANGES — update the stale “Scaffold Stellar” entry in static/llms.txt.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
docs/tools/scaffold-stellar.mdx Rewrites the toolkit overview.
docs/tools/README.mdx Updates the tools listing.
docs/tools/openzeppelin-contracts.mdx Renames the Scaffold package reference.
docs/build/smart-contracts/getting-started/README.mdx Updates the quick-start recommendation.
docs/build/smart-contracts/getting-started/hello-world-frontend.mdx Condenses frontend setup guidance.
config/theme/navbar.ts Updates the navigation label.

Comment thread docs/tools/stellar-scaffold.mdx Resolved
@zachfedor

Copy link
Copy Markdown
Contributor Author

After discussing with @oceans404, I went ahead and updated the filename/route and added a redirect rule. Let me know if I need to do anything else, though, because it looks like the check is failing.

@kaankacar

Copy link
Copy Markdown
Contributor

Verified this branch on a local build. Pagination, sidebar, breadcrumbs, and section page count match production. The nginx redirect, routes.txt, llms.txt, and navbar entries are consistent. Both crate names resolve on crates.io. All new external links work except one.

One blocker: https://stellarscaffold.org/img/code_hero.svg returns 404. The hero image on the new page renders broken. Please point it at a working asset, or commit the image to static/img and reference it locally.

The check-routes-removal failure is not a content problem. That workflow only posts a warning comment when routes.txt loses a line, and the post gets a 403 on fork PRs. Your redirect already satisfies what the warning asks for.

@kaankacar

Copy link
Copy Markdown
Contributor

🤖 Automated message from Kaan's Automated Triage Bot.

Reviewing because a review was requested by @ElliotFriend.

The hero image is still broken. https://stellarscaffold.org/img/code_hero.svg returns 404 today. That host serves only logo.svg and aha-social-card.png, so I cannot pick the replacement for you. Please give the correct asset URL, or commit the image to static/img.

Everything else verifies: the rename is complete, the route and redirect work, and every other link resolves.

@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Back on this because @zachfedor pushed a new commit.

Thanks for the update. The install steps and the scaffold.yml note match v0.0.30. The build passes. One blocker remains:

Can you merge and deploy that PR, then reply here? I will recheck the image and update the label.

(check-routes-removal fails only because the workflow cannot comment on fork PRs. Your redirect covers it. No live preview was available, so I reviewed from the diff and the build check.)

chadoh pushed a commit to stellar-scaffold/cli that referenced this pull request Oct 23, 2026
- Adds `img/scaffold-hero.png`: a terminal running `init` and `npm run
dev` beside the React template in the browser
- Unblocks the hero image on Stellar's developer docs page,
stellar/stellar-docs#2708, which currently links to a removed asset

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_01QhmaWLv4yExX8LahP5UtUb

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@zachfedor

Copy link
Copy Markdown
Contributor Author

Merged and deployed!

@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Back on this because @zachfedor replied.

Thanks! The hero image loads now, so the content is ready. One blocker remains: main moved to Mintlify in #2941, and this branch now conflicts in 9 files. I cannot push the merge for you, because it brings in workflow changes. Can you rebase onto main and port the edits?

  • Rename docs/tools/scaffold-stellar.mdx and update its path in docs.json.
  • Put the redirect in redirects.json. redirects.conf, routes.txt, static/llms.txt and navbar.ts are gone.
  • Re-apply your edits to getting-started/index.mdx (was README.mdx), hello-world-frontend.mdx, tools/index.mdx, plugins-list.mdx and openzeppelin-contracts.mdx.

MIGRATING.md lists the new conventions. (check-routes-removal is red only from the old fork 403. That workflow no longer exists on main. No live preview was available.)

@zachfedor
zachfedor force-pushed the scaffold-stellar-tools-page branch from a3bd543 to acb392e Compare October 27, 2026 13:53
@mintlify

mintlify Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Preview approval required

This pull request comes from a fork, so Mintlify won't build a preview until someone with write access approves commit 6c31502.

Approve preview

@zachfedor
zachfedor force-pushed the scaffold-stellar-tools-page branch from acb392e to 33f5c03 Compare October 27, 2026 13:54
@stellar-triage-bot stellar-triage-bot Bot added bot:ready-to-merge Bot verified and approved; waiting for a maintainer to merge and removed bot:needs-author Bot asked the author for a change or clarification labels Oct 27, 2026
@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Back on this because @zachfedor pushed the Mintlify port.

Verified: the port matches the content I checked before, the redirect and docs.json entry are correct, all links resolve, and checks is green. No live preview was available. Ready to merge.

External-origin change, so a maintainer merges this.

@ElliotFriend ElliotFriend left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for porting this to Mintlify, and for keeping the redirect from the old /docs/tools/scaffold-stellar path. One small thing on docs/tools/stellar-scaffold.mdx: the "officially sanctioned Stellar Development Foundation Public Good" line links to the grants and funding page, which doesn't say that. Could we rewrite that sentence, or link to a source that says it? Once that's sorted, I'll approve the preview run and merge.

@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Back on this because @ElliotFriend reviewed.

Done in 6c31502. The grants page does not call Scaffold a Public Good, so I removed that claim. The sentence now says it is open source under Apache 2.0 and actively maintained. Both facts come from the stellar-scaffold/cli repo. checks is green. No live preview was available. Ready to merge.

External-origin change, so a maintainer merges this.

A maintainer can build the preview: open the Mintlify Preview (fork) check and click Approve preview.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:ready-to-merge Bot verified and approved; waiting for a maintainer to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants