Document traffic rules, comments, connectors, team access, and feedback - #86

• The Zero connectors page says connector({ as }) takes only team or owner, but the code also accepts visitor (each signed-in visitor's own account). The visitor_role_not_bindable error and the ConnectorGate connect button are undocumented, so readers would conclude per-visitor integrations aren't possible. (fixed by commit c6330e8)
  _{Found by Indent Review Agent}
• The sf feedback output sample on /feedback shows two lines, but the CLI prints a third read-back hint line, and the /cli/agent-commands sample includes it. The two pages disagree. (fixed by commit c6330e8)
  _{Found by Indent Review Agent}

• Indent Review Agent · No remaining issues found

Indent Review Agent

• Summary: No remaining issues found
• Last ran on commit: c6330e8c
• Latest result

  {
    "summary": "No remaining issues found",
    "findings": []
  }

This pull request adds documentation for account features, connectors, project configuration, traffic rules, migration, Comments, feedback, and CLI commands. It also updates navigation, cross-links, and a redirect.

Changes: Team access and identity

Layer / File(s)	Summary
Team membership and onboarding content/(account)/auto-enroll.mdx, content/(account)/meta.ts, content/(concepts)/teams.mdx	The account and team guides cover allowed email domains, Enterprise subteams, and the related navigation links.
Identity connection setup and lifecycle content/(account)/identity-connections.mdx	The new guide covers OIDC and signer setup, Space grants, signer proof exchange, updates, revocation, and errors.
Enterprise subteams content/(account)/subteams.mdx	The new guide covers Enterprise eligibility, shared seats, membership rules, management workflows, and deletion constraints.

Connectors

Layer / File(s)	Summary
Connector registration and account connections content/(dynamic)/connectors.mdx, content/api/connectors.mdx, content/cli/connectors.mdx	The guides document connector discovery, registration, account connections, lifecycle operations, and CLI commands.
Policies, tools, and runs content/(dynamic)/connectors.mdx, content/api/connectors.mdx, content/api/index.mdx	The guides cover policy resolution, tool discovery, run approval and status, errors, and links to the API reference.
Space bindings and runtime calls content/(dynamic)/zero-connectors.mdx, content/(dynamic)/functions.mdx, content/(dynamic)/meta.ts, content/api/connectors.mdx, content/api/meta.ts, content/cli/meta.ts	The guides document Space bindings, Zero role declarations and calls, Worker access, call states, and navigation updates.

Project configuration and traffic rules

Layer / File(s)	Summary
Project configuration formats and discovery content/(reference)/config-file.mdx, content/cli/project.mdx, content/cli/git.mdx, content/cli/index.mdx	The guides document TypeScript and JSON config formats, discovery and precedence, initialization, and related CLI options.
Traffic rule definitions and testing content/(reference)/config-file.mdx, content/(serve)/traffic-rules.mdx	The guides describe rule syntax, matching, ordering, validation, local testing, and limits.
Traffic rule operations and serving behavior content/(serve)/traffic-rules.mdx, content/cli/rules.mdx, content/(serve)/meta.ts, content/(serve)/routing.mdx	The guides cover rule operations, serving behavior, API references, and navigation links.

Static site migration

Layer / File(s)	Summary
Capture requirements and command output content/(publish)/migrate.mdx	The guide describes command options, URL restrictions, capture limits, and publication output.
Attempt recovery and follow-up content/(publish)/migrate.mdx, content/(publish)/meta.ts, redirects.ts	The guide covers recovery, claiming and editing a copy, and errors. The page list and redirect point to the migration guide.

Comments

Layer / File(s)	Summary
Comments access and configuration content/(serve)/access.mdx, content/(serve)/comments.mdx, content/(serve)/meta.ts, content/cli/comments.mdx	The guides document prerequisites, grants, surfaces, settings, and embedded-site configuration.
Threads and collaboration content/(serve)/comments.mdx, content/cli/comments.mdx	The guides cover thread operations, reactions, status, unread tracking, spam handling, notifications, and exports.
Comments CLI settings and validation content/cli/comments.mdx, styles/Spacefast/spelling-exceptions.txt	The CLI guide covers settings, validation, and errors. The spelling exceptions list adds unreact.

Feedback

Layer / File(s)	Summary
Feedback submission and readback content/(reference)/feedback.mdx, content/(reference)/meta.ts	The guide documents CLI and API submission, sender attribution, readback, agent guidance, and limits.
Feedback CLI and error reporting references content/cli/agent-commands.mdx, content/cli/index.mdx, content/troubleshooting.mdx	The pages update feedback receipt details, add a feedback command card, and link error reports to feedback.

CLI utilities

Layer / File(s)	Summary
Shell completion content/cli/index.mdx	The CLI overview describes completion setup for supported shells and cache refresh.
Space name availability check content/cli/spaces.mdx	The Spaces reference documents name normalization, team selection, availability results, and errors.

Priority: Normal

Estimated code review effort: 4 (Complex) | ~75 minutes

Change: Other

Possibly related PRs

• spacefast/docs#83: Documents team-private Space access presets, which connect to this pull request’s team membership and allowed-domain onboarding guidance.

Suggested reviewers: alexanderaidun-a8c

Merge Risk: Low · up to ad8a8

This PR only adds and updates documentation. Copy-pasted Worker examples would run a write operation on every request and should be guarded before merge. A few pages also have wording that contradicts itself or omits steps and deadlines. Fix these before publishing to avoid misleading readers.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title names five areas (traffic rules, comments, connectors, team access, feedback) that the changeset documents. The PR objectives confirm the title matches the main change, which adds documentat…
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 8…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title names five areas (traffic rules, comments, connectors, team access, feedback) that the changeset documents. The PR objectives confirm the title matches the main change, which adds documentation pages for these shipped features and related updates. Team access is covered by the allowed email domains and Enterprise subteams docs. The title is slightly long at 71 characters but remains specific and accurate.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 8 files. (26 skipped: 26 unsupported.)

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• X
• Mastodon
• Reddit
• LinkedIn