Skip to content

claude/improve-prism-docs-VH8qW#5

Merged
andyfischer merged 1 commit into
mainfrom
claude/improve-prism-docs-VH8qW
Apr 18, 2026
Merged

claude/improve-prism-docs-VH8qW#5
andyfischer merged 1 commit into
mainfrom
claude/improve-prism-docs-VH8qW

Conversation

@andyfischer
Copy link
Copy Markdown
Contributor

  • prism-framework-ui README: rewrite to match the actual exports; the
    previous README referenced a nonexistent QueryProvider, a missing
    React Query integration, and unexported Radix UI components.
  • Clarify the /api/ path prefix rule: HTTP callers (including
    prism call, webFetch, and prism generate-api-clients) need /api/
    in the URL, but createEndpoint({ path }) must not include it. Fixed
    the README, endpoint-tools, source-directory-organization, and
    vite-setup examples accordingly.
  • Fix the creating-mobile-apps cross-platform comparison table: desktop
    auth uses the getAuth option, not Express middleware.
  • Fix the MigrationBehavior values listed in launch-configuration and
    database-setup to match the real values (strict, safe-upgrades,
    full-destructive-updates, ignore).
  • Correct authorization to handle the
    getCurrentRequestContext() === undefined case and describe what
    requires: ['authenticated-user'] actually does (metadata only; the
    base framework does not auto-reject).
  • generate-api-clients-config: reference /api/openapi.json and the
    port-assignment-based port resolution.
  • Expand overview and the README doc index to mention previously
    undocumented guides (creating-mobile-apps, cors-setup, error-handling,
    generate-api-clients-config, metrics, source-directory-organization).
  • vite-setup: proxy only /api, and document the matching webFetch
    usage that includes /api in the path (or in baseUrl).

- prism-framework-ui README: rewrite to match the actual exports; the
  previous README referenced a nonexistent QueryProvider, a missing
  React Query integration, and unexported Radix UI components.
- Clarify the /api/ path prefix rule: HTTP callers (including
  `prism call`, webFetch, and `prism generate-api-clients`) need /api/
  in the URL, but `createEndpoint({ path })` must not include it. Fixed
  the README, endpoint-tools, source-directory-organization, and
  vite-setup examples accordingly.
- Fix the creating-mobile-apps cross-platform comparison table: desktop
  auth uses the `getAuth` option, not Express middleware.
- Fix the MigrationBehavior values listed in launch-configuration and
  database-setup to match the real values (`strict`, `safe-upgrades`,
  `full-destructive-updates`, `ignore`).
- Correct authorization to handle the
  `getCurrentRequestContext() === undefined` case and describe what
  `requires: ['authenticated-user']` actually does (metadata only; the
  base framework does not auto-reject).
- generate-api-clients-config: reference `/api/openapi.json` and the
  `port-assignment`-based port resolution.
- Expand overview and the README doc index to mention previously
  undocumented guides (creating-mobile-apps, cors-setup, error-handling,
  generate-api-clients-config, metrics, source-directory-organization).
- vite-setup: proxy only `/api`, and document the matching webFetch
  usage that includes `/api` in the path (or in `baseUrl`).
@andyfischer andyfischer merged commit 8912a70 into main Apr 18, 2026
3 of 4 checks passed
@andyfischer andyfischer deleted the claude/improve-prism-docs-VH8qW branch April 18, 2026 01:55
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.

2 participants