backnotprop / backnotprop/plannotator

Root build script runs dependent targets out of order

Open Beginner friendly
#1,388 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
8.7k
Forks
649
Avg merge
11h 12m
Merged PRs (30d)
109

Description

## What happened?

On a clean checkout, `bun run build` fails in `apps/hook` because its build copies `../review/dist/index.html` before the review application has been built.

## Plannotator version

0.25.1 from the local `rnoz/integration` checkout.

## OS

Linux x86_64 (tested via Bun on a Linux userspace host).

## Agent

Reproduced through the repository build scripts with Bun 1.4.0.

## Where did it happen?

Repository root `package.json`, `scripts.build`, invoking `apps/hook/package.json`.

## Reproduction

Run `bun install --frozen-lockfile` followed by `bun run build` from the repository root with no `apps/review/dist/index.html` present.

## Observed result

`build:hook` reports `cp: cannot stat '../review/dist/index.html': No such file or directory` and the root build exits non-zero.

## Expected result

The root build should build the review bundle before the hook bundle and complete from a clean checkout.

## Root cause

The root `build` script invokes `build:hook` before `build:review`, although `apps/hook/package.json` has a runtime build dependency on the review output.

## Proposed scope

Change only the root build sequence to `bun run build:review && bun run build:hook && bun run build:opencode`; keep the existing focused scripts unchanged.

## GitHub overlap check

Authenticated `gh issue list --repo backnotprop/plannotator --state all --search "build order"` and `gh pr list --repo backnotprop/plannotator --state all --search "build review before hook"` returned no matching open or closed issue or PR on 2026-08-25.

Contributor guide

Open the contributing guide

Research direction

Start with the root package.json scripts.build and compare it with apps/hook/package.json, especially the dependency on ../review/dist/index.html. Run bun install --frozen-lockfile and bun run build from a clean checkout. Done means the review bundle is built before the hook bundle and the root build completes successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
bun, typescript
Domain
build-system
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
92/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.