Skip to content

Add guidance for adding Lakebase and Model Serving to existing apps#81

Merged
pkosiec merged 2 commits into
mainfrom
pkosiec/lakebase-handle-existing-apps
May 20, 2026
Merged

Add guidance for adding Lakebase and Model Serving to existing apps#81
pkosiec merged 2 commits into
mainfrom
pkosiec/lakebase-handle-existing-apps

Conversation

@pkosiec
Copy link
Copy Markdown
Member

@pkosiec pkosiec commented May 19, 2026

Summary

  • lakebase.md: Replaced "do not add Lakebase manually" blocker with a full "Adding Lakebase to an Existing App" section (following the genie.md pattern) — covers databricks.yml, app.yaml, server.ts, and .env changes
  • model-serving.md: Major rewrite — replaced outdated tRPC + getExecutionContext() pattern with the modern serving() plugin API, including "Adding to Existing App" section, useServingStream/useServingInvoke hooks, HTTP endpoints, and named endpoint aliases
  • SKILL.md: Updated routing table to reference serving() plugin instead of tRPC
  • trpc.md: Added serving/jobs to plugin checklist, replaced serving endpoint example with generic API example, updated decision tree to prefer plugins over tRPC
  • overview.md: Updated Model Serving init command and decision tree references

All patterns verified against appkit source code (manifests, templates, plugin implementations, and official docs).

Test plan

  • Verify python3 scripts/skills.py validate passes
  • Review lakebase.md "Adding to Existing App" section matches genie.md pattern
  • Review model-serving.md matches appkit docs for serving plugin API
  • Verify no stale getExecutionContext + servingEndpoints.query patterns remain in model-serving or SKILL.md contexts

This pull request and its description were written by Isaac.

…sting apps

- lakebase.md: Replace "do not add manually" blocker with "Adding Lakebase
  to an Existing App" section (databricks.yml, app.yaml, server.ts, .env)
- model-serving.md: Rewrite to use serving() plugin API instead of outdated
  tRPC + getExecutionContext() pattern; add "Adding to Existing App" section,
  frontend hooks (useServingStream, useServingInvoke), HTTP endpoints, and
  named endpoint aliases
- SKILL.md: Update routing table to reference serving() plugin
- trpc.md: Add serving/jobs to plugin checklist, replace serving endpoint
  example with generic API example, update decision tree
- overview.md: Update Model Serving init command and decision tree references

Co-authored-by: Isaac
@pkosiec pkosiec changed the title docs(apps): add guidance for adding Lakebase and Model Serving to existing apps Add guidance for adding Lakebase and Model Serving to existing apps May 20, 2026
- lakebase.md: Remove postgres from user_api_scopes default example
  (only needed for OBO, not basic usage — matches appkit template)
- lakebase.md, model-serving.md: Remove npm run sync from closing
  paragraphs (runs automatically during dev/build)

Co-authored-by: Isaac
@pkosiec pkosiec marked this pull request as ready for review May 20, 2026 13:59
@pkosiec pkosiec requested a review from a team as a code owner May 20, 2026 13:59
@pkosiec pkosiec merged commit bf6d932 into main May 20, 2026
1 check passed
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