fix: disinto release uses undefined PROJECT_REPO variable #166

New issue

Closed

opened 2026-04-03 06:09:50 +00:00 by dev-bot · 0 comments

dev-bot commented

2026-04-03 06:09:50 +00:00

Collaborator

Problem

bin/disinto line 2912 references ${PROJECT_REPO} which is never set, causing:

./bin/disinto: line 2911: PROJECT_REPO: unbound variable

The vault PR should be created on the ops repo, not the code repo.

Proposed solution

Replace ${PROJECT_REPO} with ${FORGE_OPS_REPO} at line 2912 and 2922.

Also ensure FORGE_OPS_REPO is loaded from projects/*.toml by lib/env.sh (currently ops_repo is defined in the project TOML but not exported as an env var).

Affected files

bin/disinto (lines 2912, 2922)
lib/env.sh or lib/load-project.sh (needs to export FORGE_OPS_REPO)

Acceptance criteria

disinto release v0.x.0 creates PR on ops repo without errors
FORGE_OPS_REPO is set when project config is loaded

Dependencies

Depends on #170

## Problem `bin/disinto` line 2912 references `${PROJECT_REPO}` which is never set, causing: ``` ./bin/disinto: line 2911: PROJECT_REPO: unbound variable ``` The vault PR should be created on the **ops repo**, not the code repo. ## Proposed solution Replace `${PROJECT_REPO}` with `${FORGE_OPS_REPO}` at line 2912 and 2922. Also ensure `FORGE_OPS_REPO` is loaded from `projects/*.toml` by `lib/env.sh` (currently `ops_repo` is defined in the project TOML but not exported as an env var). ## Affected files - `bin/disinto` (lines 2912, 2922) - `lib/env.sh` or `lib/load-project.sh` (needs to export FORGE_OPS_REPO) ## Acceptance criteria - [ ] `disinto release v0.x.0` creates PR on ops repo without errors - [ ] `FORGE_OPS_REPO` is set when project config is loaded ## Dependencies Depends on #170