Skip to content

feat: Build studio#20

Merged
kriszyp merged 1 commit into
mainfrom
build-studio
Mar 3, 2026
Merged

feat: Build studio#20
kriszyp merged 1 commit into
mainfrom
build-studio

Conversation

@dawsontoth
Copy link
Copy Markdown
Contributor

This clones and builds and copies Studio as a part of the build process. It's not clear to me if we publish from the root directory, or from the core directory, though. And I don't know if core will actually look for the studio folder for its static files. I think I need to mv it into core? Probably?

Screenshot 2026-02-27 at 12 55 52 PM

@dawsontoth dawsontoth marked this pull request as ready for review February 27, 2026 20:30
@dawsontoth dawsontoth requested review from a team as code owners February 27, 2026 20:30
Comment thread build-tools/build-studio.sh Outdated
Copy link
Copy Markdown
Member

@kriszyp kriszyp left a comment

Choose a reason for hiding this comment

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

Thank you!

@kriszyp kriszyp merged commit 24aa71c into main Mar 3, 2026
20 of 23 checks passed
@kriszyp kriszyp deleted the build-studio branch March 3, 2026 13:14
heskew added a commit that referenced this pull request May 19, 2026
…bel-gated review

Three changes from ai-review-prompts main since this PR was first opened:

  - #37: harper/common.md gains "Meta-checks" section + reuse /
    CI hygiene / lockfile drift bullets.
  - #38: label-gated review for bot-authored PRs (`claude-review`
    label as opt-in gesture).
  - #40: revert of #39's workflow-level `permissions: {}` on
    reusables — #39 broke reusable callers via the calling-
    workflow-caps-reusable rule. Empirically verified working
    on harper main (PR #576).

Caller changes in this commit:

  - Pin bumped to `f22bf7d` (post-#37 + post-#38 + post-#40).
    Catches the pin up from `3278ce4e` (post-#20) which main
    was rolled back to during the #39 incident.
  - `pull_request: types:` adds `labeled` so the new bot-PR
    gesture fires.
  - **No caller-side `permissions: {}`**. Earlier version
    included that recommendation from #39; #40 reverted the
    recommendation.

Repo-side prerequisite (already applied via `gh label create`):

  - `claude-review` label exists on this repo.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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.

3 participants