docs: add small-team Instance tab state checklist

[YuuGR1337]

Fixes #269

---

Summary by CodeRabbit

• Documentation
  • Added documentation outlining UI state specifications for the Instance tab, including single-user defaults, admin workflows, approval processes, user visibility, loading states, access control, and error messaging scenarios.

[github-actions]

PR Context Summary

• Linked issue(s): docs: add small-team Instance tab state checklist #269
• Referenced but not closing: none
• PR author trusted by GitHub: no
• Dependabot PR: no
• PR assignee synced from linked issue: no change
• Macroscope review status: Skipped because do-not-merge is present.

Suggested issue links

• No strong issue match found yet.

Use Fixes #123 or Closes #123 in the PR body when one of the suggestions is the intended issue.
Manual rerun: Actions > PR Context Triage > Run workflow > set pr_number and force_review=true.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR adds an "Instance Tab UI State Checklist" section to the small-team authentication planning document. The checklist specifies expected UI states for the Instance tab, including single-user defaults, admin setup flows, join request handling, pending approvals, active users visibility, loading states, unauthorized access, and backend error messaging.

Changes

Instance Tab UI States Documentation

Layer / File(s)	Summary
Instance Tab UI State Checklist docs/plans/not-started/small-team-authentication.md	Checklist enumerates expected Instance tab states covering admin vs non-admin visibility, instance creation and join flows, pending approvals and active users display, and loading/error/unauthorized handling behaviors.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

• feat: add Instance tab for small-team sharing flows #263: Parent issue for the Instance tab feature; this PR documents the UI states that the parent issue references for implementation.

Poem

🐰 A checklist hops into the plan,
States and flows now mapped with care,
Admin paths and joiners too,
UI blueprints bright and true,
Implementation awaits its turn! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is minimal but references the linked issue #269. However, it lacks the required template sections such as Summary, Type of change, What changed, How to test, and the mandatory checklists.	Fill out the complete PR description template including Summary, Type of change, What changed section, How to test, and all required checklist items.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title accurately and concisely summarizes the main change: adding documentation for an Instance tab state checklist to the small-team authentication plan.
Linked Issues check	✅ Passed	The code changes add the Instance tab UI state checklist covering admin/joining-user flows, single-user default behavior, loading states, and error handling as required by issue #269.
Out of Scope Changes check	✅ Passed	All changes are scoped to documentation updates of the small-team authentication plan file, directly addressing the issue requirements without implementing frontend UI.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[macroscopeapp]

Approvability

Verdict: Approved

Pure documentation change adding a UI state checklist to a planning markdown file. No code or runtime behavior is affected.

^{You can customize Macroscope's approvability policy. Learn more.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/plans/not-started/small-team-authentication.md`:
- Around line 43-50: The checklist currently covers flows but omits explicit
Empty and Success states; update the checklist by adding two dedicated items:
"**Empty**: no instances or invites exist — show the empty state and CTA for
creating or inviting" and "**Success**: instance created/joined — show the
running instance UI and user/role list", placing them alongside the existing
items (e.g., near "Admin setup", "Active users", "Loading", "Unauthorized",
"Error") so acceptance criteria explicitly include empty and success states for
implementation and QA.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4e3605d9-eca3-4914-8a93-83f54806a643

📥 Commits

Reviewing files that changed from the base of the PR and between bf7a916 and ea5328f.

📒 Files selected for processing (1)

• docs/plans/not-started/small-team-authentication.md

[coderabbitai]

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Add explicit Empty and Success checklist states to match acceptance criteria.

Line 43-50 currently implies those states via flow bullets, but the issue objective asks for explicit state coverage (empty, loading, unauthorized, error, success). Please add dedicated Empty and Success checklist items so the doc is unambiguous for implementation and QA.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/plans/not-started/small-team-authentication.md` around lines 43 - 50,
The checklist currently covers flows but omits explicit Empty and Success
states; update the checklist by adding two dedicated items: "**Empty**: no
instances or invites exist — show the empty state and CTA for creating or
inviting" and "**Success**: instance created/joined — show the running instance
UI and user/role list", placing them alongside the existing items (e.g., near
"Admin setup", "Active users", "Loading", "Unauthorized", "Error") so acceptance
criteria explicitly include empty and success states for implementation and QA.