Skip to content

docs: replace stale getAdminContext snippet in packages/ui.md#612

Merged
borisno2 merged 1 commit into
mainfrom
claude/issue-563-kpvbq2
Jun 27, 2026
Merged

docs: replace stale getAdminContext snippet in packages/ui.md#612
borisno2 merged 1 commit into
mainfrom
claude/issue-563-kpvbq2

Conversation

@borisno2

Copy link
Copy Markdown
Member

Summary

  • getAdminContext from @opensaas/stack-ui/server does not exist — the server export is type-only (export {} with only ServerActionInput/ActionResult re-exports)
  • All occurrences of getAdminContext in docs/content/packages/ui.md have been replaced with the correct host-supplies-context pattern from examples/starter-auth
  • The correct pattern: import getContext/config from the generated .opensaas/context, resolve the session in the host page, and pass a 'use server' wrapper around context.serverAction to <AdminUI />
  • Updated sections: Quick Start, Package Exports, AdminUI, ItemCreateForm, ItemEditForm, ListTable, Server Utilities, Performance, Custom Dashboard example
  • The Server Utilities section now documents ServerActionInput (the actual export) and explicitly notes there is no getAdminContext helper

Test plan

  • All getAdminContext import/usage snippets replaced with getContext + getSession pattern
  • Server Utilities section documents ServerActionInput type instead of non-existent getAdminContext
  • Code snippets match the pattern in examples/starter-auth/app/admin/[[...admin]]/page.tsx

Closes #563


Generated by Claude Code

…n in packages/ui.md

`getAdminContext` from `@opensaas/stack-ui/server` does not exist — the
server export is type-only. Replace every occurrence with the correct
starter-auth pattern: import `getContext`/`config` from `.opensaas/context`,
resolve the session in the host page, and pass a `'use server'` wrapper
around `context.serverAction` to `<AdminUI />`.

Closes #563
@changeset-bot

changeset-bot Bot commented Jun 27, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6410829

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel

vercel Bot commented Jun 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
stack-docs Ready Ready Preview, Comment Jun 27, 2026 10:15pm

@borisno2 borisno2 enabled auto-merge (squash) June 27, 2026 22:19
@borisno2 borisno2 merged commit 2398c9f into main Jun 27, 2026
6 checks passed
@borisno2 borisno2 deleted the claude/issue-563-kpvbq2 branch June 27, 2026 22:21
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for Core Package Coverage (./packages/core)

Status Category Percentage Covered / Total
🔵 Lines 91.51% (🎯 65%) 841 / 919
🔵 Statements 90.72% (🎯 65%) 871 / 960
🔵 Functions 97.93% (🎯 62%) 142 / 145
🔵 Branches 79.05% (🎯 50%) 551 / 697
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for UI Package Coverage (./packages/ui)

Status Category Percentage Covered / Total
🔵 Lines 76.03% 92 / 121
🔵 Statements 75.39% 95 / 126
🔵 Functions 75.6% 31 / 41
🔵 Branches 65.78% 75 / 114
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for CLI Package Coverage (./packages/cli)

Status Category Percentage Covered / Total
🔵 Lines 79.21% 1490 / 1881
🔵 Statements 78.92% 1550 / 1964
🔵 Functions 84.45% 201 / 238
🔵 Branches 67.25% 653 / 971
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for Auth Package Coverage (./packages/auth)

Status Category Percentage Covered / Total
🔵 Lines 74.64% 159 / 213
🔵 Statements 69.74% 166 / 238
🔵 Functions 83.11% 64 / 77
🔵 Branches 70.67% 94 / 133
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for Storage Package Coverage (./packages/storage)

Status Category Percentage Covered / Total
🔵 Lines 73.22% 186 / 254
🔵 Statements 75% 207 / 276
🔵 Functions 83.33% 65 / 78
🔵 Branches 70.73% 174 / 246
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for RAG Package Coverage (./packages/rag)

Status Category Percentage Covered / Total
🔵 Lines 47.97% 355 / 740
🔵 Statements 48.14% 377 / 783
🔵 Functions 54.26% 70 / 129
🔵 Branches 42.55% 180 / 423
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for Storage S3 Package Coverage (./packages/storage-s3)

Status Category Percentage Covered / Total
🔵 Lines 100% 40 / 40
🔵 Statements 100% 40 / 40
🔵 Functions 100% 9 / 9
🔵 Branches 100% 19 / 19
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for Storage Vercel Package Coverage (./packages/storage-vercel)

Status Category Percentage Covered / Total
🔵 Lines 100% 38 / 38
🔵 Statements 100% 38 / 38
🔵 Functions 100% 8 / 8
🔵 Branches 100% 22 / 22
File CoverageNo changed files found.
Generated in workflow #1207 for commit 6410829 by the Vitest Coverage Report Action

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.

docs: packages/ui.md still shows getAdminContext from '@opensaas/stack-ui/server', which does not exist

2 participants