Skip to content

docs(cli): correct static build deployment examples - #466

Open
SaKaNa-Y wants to merge 1 commit into
devframes:mainfrom
SaKaNa-Y:docs/static-build-base
Open

SaKaNa-Y wants to merge 1 commit into
devframes:mainfrom
SaKaNa-Y:docs/static-build-base

Conversation

@SaKaNa-Y

Copy link
Copy Markdown
Contributor

Summary

The standalone CLI guide and CAC adapter page show build --base for deployment under a custom URL path. That command exits with an unknown-option error for --base because #234 removed the flag.

Remove both invalid examples and explain how to deploy the output with relative SPA asset paths.

- my-tool build --out-dir dist-static --base /tool/
+ my-tool build --out-dir dist-static

Evidence

  • Both old examples reject --base. The corrected command exports successfully. A public client reads the static snapshot at /tool/ and /my-tool/, and relative asset requests return HTTP 200. This is an HTTP/client check, not a full browser UI test.
  • pnpm lint, pnpm knip, CI=1 pnpm test, pnpm typecheck, and pnpm build passed. The test gate passed after regenerating stale local declarations.

Merge Danger

Door: two-way. Documentation only.

Blast Radius: documentation. CLI behavior and build output stay unchanged.

@vercel

vercel Bot commented Oct 28, 2026

Copy link
Copy Markdown

@SaKaNa-Y is attempting to deploy a commit to the NuxtLabs Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

👁️‍🗨️ Review this pull request with grouped, summarized diffs at:
👉 https://pulls.review/gh/devframes/devframe/466?from=github-actions[bot]

raw result

💭 analyzed by vercel-ai-gateway/anthropic/claude-sonnet-5
🕰️ 2026-10-28 03:55 UTC
🔗 head 6f5787c
🤖 automated by pulls.review

{
  "headSha": "6f5787c24c934c3b7e7c0adba24e67478d3cc29f",
  "result": {
    "overallSummary": "Documentation-only fix correcting two CLI deployment examples that referenced a `--base` flag no longer supported, replacing them with guidance on using relative SPA asset paths for custom deployment paths.\"",
    "groups": [
      {
        "key": "docs-fix",
        "label": "Fix CLI build docs",
        "summary": "Removes the invalid `build --base` examples (the flag was removed in #234) and instead explains deploying static output under a custom path using relative asset paths (`vite.base: './'`).",
        "category": "docs",
        "filePaths": [
          "docs/content/1.guide/10.standalone-cli.md",
          "docs/content/2.adapters/2.cac.md"
        ]
      }
    ],
    "schemaVersion": 1,
    "source": "llm",
    "generatedAt": "2026-10-28T03:55:38.910Z",
    "model": "vercel-ai-gateway/anthropic/claude-sonnet-5",
    "locale": "en"
  }
}

@SaKaNa-Y
SaKaNa-Y marked this pull request as ready for review October 29, 2026 13:23

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant