Skip to content

Document team Design Systems - #67

Open
alexanderaidun-a8c wants to merge 3 commits into
mainfrom
codex/docs-design-system
Open

alexanderaidun-a8c wants to merge 3 commits into
mainfrom
codex/docs-design-system

Conversation

@alexanderaidun-a8c

@alexanderaidun-a8c alexanderaidun-a8c commented Oct 8, 2026 •

Copy link
Copy Markdown

Summary

  • Add a Design System guide that explains why teams save shared standards, then covers setup, imports and starter kits, brand files, applying standards to a named project, and changing or reverting saved rules.
  • Clarify the agent workflow: importing does not change a site, applying is scoped to the requested project, and existing Spaces update only when named.
  • Add the guide to Concepts navigation and link it from the Teams and Agents pages. Update Skills to distinguish the base spacefast skill from the four Design System task skills.

Verification

  • bun run check and Vale suggestion check passed after the final copy edit.
  • bun run validate, bun run build, bun run audit, bun run verify:routes, bun run verify:public-safety, and bun run verify:prose passed during branch preparation.
  • git diff --check origin/main...HEAD passed.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@indent

indent Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Issues

1 potential issue found:

  • The docs say Spacefast plugins include four Design System skills (import, apply, verify, update-design). The published spacefast/plugins release (0.5.1) contains none of them, so readers who install the plugin today won't find them.
    Found by Indent Review Agent

↑ Select any checkbox above to have Indent auto-fix the issue

CI Checks

All required checks pass. The non-required Spacefast Builds check fails with "Static deploys cannot upload runtime control paths: storage/index.html", and it fails the same way on main.

Failing Spacefast Builds
  • The Spacefast preview deploy rejects the built storage/index.html as a runtime control path. This PR doesn't touch any storage page, and the latest main commit shows the same failure, so it predates this change.

Review agents

Select any unchecked box below to run or rerun that agent.

Found issues (1)
  • Indent Review Agent · One bug: the docs list four Design System skills that the published plugins don't include yet.
Full results

Indent Review Agent

  • Summary: One bug: the docs list four Design System skills that the published plugins don't include yet.
  • Last ran on commit: 77233efe
  • Latest result
    {
      "summary": "One bug: the docs list four Design System skills that the published plugins don't include yet.",
      "findings": [
        {
          "fix": "Hold this PR until the plugin release that adds the four skills is published to spacefast/plugins. Or, for now, say the skills ship with the next plugin release, and give the import step in design-system.mdx the same 'when available' wording the apply section uses.",
          "line": 23,
          "path": "content/agents/skills.mdx",
          "issue": "skills.mdx says \"Spacefast plugins include four skills for the Design System\" and the intro says the plugins \"also include task skills\". design-system.mdx (Start a Design System) also says the `import-design-system` skill does the import, with no caveat. I checked the public spacefast/plugins repo at HEAD (d2cfc0f, release-manifest version 0.5.1, generated 2026-10-08). `claude-code/skills`, `codex/skills`, `cursor/skills`, and the root `skills/` hold only build-website, custom-domain, edit-space, fix-deployment, setup, share-space, and spacefast. None of import-design-system, apply-design-system, verify-design-system, or update-design are there. They exist only in the unreleased monorepo plugin source (version 0.6.0). The dashboard's own Copy prompt already allows for this: \"If `import-design-system` isn't available, follow the documented design-system operations through MCP\". Anyone who installs the plugin today using these docs gets none of the four skills, which breaks the repo rule to document only shipped, publicly verifiable behavior.",
          "title": "Docs say the plugins ship Design System skills, but the published plugins don't include them yet",
          "severity": "bug"
        }
      ]
    }

@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: bd2b920e-6282-4aca-a2ff-aafa8f353e31
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Comment thread content/agents/skills.mdx
## What triggers it
## Design System task skills

Spacefast plugins include four skills for the [Design System](/design-system):

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.

🟡 Bug · Docs say plugins ship Design System skills, but published plugins don't include them yet

This says Spacefast plugins include four Design System skills, but the published spacefast/plugins repo (HEAD d2cfc0f, release-manifest 0.5.1) only ships build-website, custom-domain, edit-space, fix-deployment, setup, share-space and spacefast. import-design-system, apply-design-system, verify-design-system and update-design exist only in the unreleased monorepo plugin source (0.6.0). Anyone installing the plugin today gets none of them, which breaks the repo rule to document only shipped, publicly verifiable behavior.

Hold the PR until the plugin release is published, or reword to say the skills ship with an upcoming release. Also give the import step in design-system.mdx the same 'when available' wording the apply section uses.

Found by Indent Review Agent

This branch had an error being deployed

1 failed deployment
preview/codex/docs-design-system — 77233efe Deployed Oct 8, 2026 by spacefast[bot]
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