Skip to content

feat(appkit): agent-browser skill#63

Open
atilafassina wants to merge 2 commits intomainfrom
agent-browser-appkit
Open

feat(appkit): agent-browser skill#63
atilafassina wants to merge 2 commits intomainfrom
agent-browser-appkit

Conversation

@atilafassina
Copy link
Copy Markdown
Contributor

Summary

Adds an agent-browser-appkit skill to instruct users/LLMs how to visualize an AppKit app in 3 scenarios

  1. Local development (serving via Vite server)
  2. local proxy (testing local app FE against a real production env)
  3. authenticating on Databricks and storing the credentials in the agent-browser session

Documentation safety checklist

  • Examples use least-privilege permissions (no unnecessary ALL PRIVILEGES, admin tokens, or broad scopes)
  • Elevated permissions are explicitly called out where required
  • Sensitive values are obfuscated (placeholder workspace IDs, URLs, no real tokens)
  • No insecure patterns introduced (e.g. disabled TLS verification, hardcoded credentials)

Driving a deployed app with `--headed --profile <dir>` is now the
primary path; `npm run dev` is the simple local UI iteration path;
`databricks apps run-local` is demoted to an edge case for testing
platform header-injection behavior locally. Also replaces the vague
"install per the team's instructions" punt with concrete
`npm install -g agent-browser` (and pnpm/yarn equivalents) in the
pre-flight check.

Co-authored-by: Isaac
@atilafassina atilafassina requested a review from keugenek May 4, 2026 21:08
@atilafassina atilafassina marked this pull request as ready for review May 4, 2026 21:08
@atilafassina atilafassina requested a review from a team as a code owner May 4, 2026 21:08
Copilot AI review requested due to automatic review settings May 4, 2026 21:08
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new AppKit documentation for using agent-browser to visually inspect and drive Databricks Apps in three modes (deployed app URL, local Vite dev server, and databricks apps run-local proxy), and surfaces that guidance from the existing Apps skill docs.

Changes:

  • Added a new AppKit reference guide: references/appkit/agent-browser.md.
  • Linked the new guide from the databricks-apps skill index and AppKit overview reference table.
  • Updated manifest.json to include the new reference file (and updated timestamps).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
skills/databricks-apps/SKILL.md Adds a “Driving the running app in a browser” entry pointing to the new agent-browser guide.
skills/databricks-apps/references/appkit/overview.md Adds agent-browser to the AppKit references table to route readers to the new guidance.
skills/databricks-apps/references/appkit/agent-browser.md New guide covering deployed auth, local Vite iteration, and run-local proxy behavior/gotchas.
manifest.json Adds the new reference file under databricks-apps and updates updated_at fields.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

| When | Verify what's actually in staging/prod | Iterate on UI code with HMR | Test how the app behaves under the platform's auth-header proxy without deploying |
| Auth | Real OAuth/SSO | None | None — proxy injects fake `X-Forwarded-*` |
| Run command | (already deployed) | `npm run dev` | `databricks apps run-local --entry-point app.yaml` |
| agent-browser command | `agent-browser --headed --profile <dir> open <url>` (first run); plain `agent-browser open <url>` after | `agent-browser open http://localhost:<vite-port>` | `agent-browser open http://localhost:8001` |
# Subsequent runs — same profile, no login prompt:
agent-browser --profile ./.agent-browser-profile open <deployed-url>
```

Comment thread manifest.json
Comment on lines 4 to 16
"skills": {
"databricks-apps": {
"version": "0.1.1",
"description": "Databricks Apps development and deployment (evaluates analytics vs synced tables data access)",
"experimental": false,
"updated_at": "2026-04-30T11:00:26Z",
"updated_at": "2026-04-30T12:02:15Z",
"files": [
"SKILL.md",
"agents/openai.yaml",
"assets/databricks.png",
"assets/databricks.svg",
"references/appkit/agent-browser.md",
"references/appkit/appkit-sdk.md",
Comment thread manifest.json
Comment on lines 33 to 38
"databricks-core": {
"version": "0.1.0",
"description": "Core Databricks skill for CLI, auth, and data exploration",
"experimental": false,
"updated_at": "2026-04-23T13:47:44Z",
"updated_at": "2026-04-30T11:30:47Z",
"files": [
andrelandgraf pushed a commit to databricks/devhub that referenced this pull request May 9, 2026
…84)

Extends dev-guidelines.md (concatenated into every /api/bootstrap-prompt
response and the homepage Copy-prompt button payload) with a new step 8:
drive the deployed app end to end with `agent-browser`, loop on log
inspection and redeploys, and only report done after a clean pass.

Routes agents to the `databricks-apps` skill's agent-browser reference
when available, with an inline `npm install -g agent-browser` fallback
for agents that don't have the skill loaded.

Pending databricks/databricks-agent-skills#63 (adds the agent-browser
reference to the `databricks-apps` skill).
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