Skip to content

chore(bots): reviewer-bot foundation — shared SDK engine + install#832

Open
eric-wang-1990 wants to merge 1 commit into
mainfrom
eric-wang-1990/stack/reviewer-bot-foundation
Open

chore(bots): reviewer-bot foundation — shared SDK engine + install#832
eric-wang-1990 wants to merge 1 commit into
mainfrom
eric-wang-1990/stack/reviewer-bot-foundation

Conversation

@eric-wang-1990
Copy link
Copy Markdown

@eric-wang-1990 eric-wang-1990 commented Jun 4, 2026

🥞 Stacked PR

Use this link to review incremental changes.


What type of PR is this?

First of a stacked series migrating the PR-review bot from
databricks-driver-test to this repo. This PR adds only the task-agnostic
foundation (no bot logic, nothing runs on PRs yet):

  • scripts/shared/ — the common agent engine (Claude Agent SDK loop/transport,
    security guard, markers, threads, github/git ops). Self-contained; 94 unit
    tests pass under scripts/shared/tests.
  • scripts/init.py — makes scripts importable for python -m scripts.*.
  • scripts/requirements-sdk.txt — the claude-agent-sdk pin.
  • .github/actions/setup-claude-sdk + setup-jfrog — install the SDK (pip) + CLI
    (npm) through Databricks' internal JFrog mirror (the protected runner is
    egress-blocked from pypi.org/npmjs.org).
  • .github/workflows/sdk-smoke.yml — manual smoke verifying the SDK/CLI install
    on this repo's runner.

Co-authored-by: Isaac
Signed-off-by: Eric Wang e.wang@databricks.com

Description

How is this tested?

  • Unit tests
  • E2E Tests
  • Manually
  • N/A

Related Tickets & Documents

First of a stacked series migrating the PR-review bot from
databricks-driver-test to this repo. This PR adds only the task-agnostic
foundation (no bot logic, nothing runs on PRs yet):

- scripts/shared/ — the common agent engine (Claude Agent SDK loop/transport,
  security guard, markers, threads, github/git ops). Self-contained; 94 unit
  tests pass under scripts/shared/tests.
- scripts/__init__.py — makes `scripts` importable for `python -m scripts.*`.
- scripts/requirements-sdk.txt — the claude-agent-sdk pin.
- .github/actions/setup-claude-sdk + setup-jfrog — install the SDK (pip) + CLI
  (npm) through Databricks' internal JFrog mirror (the protected runner is
  egress-blocked from pypi.org/npmjs.org).
- .github/workflows/sdk-smoke.yml — manual smoke verifying the SDK/CLI install
  on this repo's runner.

Co-authored-by: Isaac
Signed-off-by: Eric Wang <e.wang@databricks.com>
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.

1 participant