databrickslabs / databrickslabs/coding-agents-databricks-apps

Core app reliability — auth, session management, gateway, .databricksignore

Open
#10 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
40
Forks
11
Avg merge
1m
Merged PRs (30d)
1

Description

Summary

Bundle of reliability fixes that have been sitting in datasciencemonkey PR #138:

  • Case-insensitive email auth — Databricks SSO headers can deliver mixed-case email; current strict comparison locks legitimate users out.
  • 30-char app-name truncation — long app names break the databricks apps get lookup and the deploy script silently fails.
  • AI Gateway auto-discovery from DATABRICKS_WORKSPACE_ID with reachability probe before use; fall back to direct serving endpoints if unreachable.
  • MAX_CONCURRENT_SESSIONS cap (default 5) to prevent resource exhaustion on a single-container app.
  • Fresh proxy token on PAT rotation — currently emits 401s until the next app restart.
  • app_state.py — lightweight persistence for owner + last rotation under ~/.coda/.
  • .databricksignore — exclude .venv/ and caches from databricks sync (huge upload otherwise).

Branch

fix/core-app-reliability — about to be pushed to this repo.

Diff scope

+1982 / -429, 22 files; new tests for gateway discovery, session limit, clipboard addon.

Notes

  • Bundled because the fixes touch overlapping surface area (auth, session lifecycle, gateway routing) and tests were written holistically. Splitting into per-fix PRs would force ugly test reshuffles.
  • Hadn't migrated when the repo moved here from datasciencemonkey.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The existing fix is described in datasciencemonkey PR #138 and branch fix/core-app-reliability; review that diff first, including app_state.py and .databricksignore. Run the new gateway-discovery, session-limit, and clipboard-addon tests to understand the current coverage. Done means the listed auth, naming, gateway, session, token, persistence, and sync behaviors are covered in this repository and the relevant tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
authentication, backend, cloud, devops
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.