Skip to content

Commit ff8ab46

Browse files
hyperpolymathclaude
andcommitted
fix(ingest): serialize ingests + fix scan-all REPOS_DIR path leak
scan-all.sh: `/var$REPOS_DIR` concatenated a literal `/var` prefix, producing `/var$REPOS_DIR` when REPOS_DIR was unset. Replace with a proper guard: use REPOS_DIR only when it is non-empty and points to an existing directory. ingest.yml: concurrent repository_dispatch deliveries race on the commit+push; one writer loses. Add a concurrency group (cancel-in-progress: false) to queue them instead. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent d184b7f commit ff8ab46

2 files changed

Lines changed: 9 additions & 2 deletions

File tree

.github/workflows/ingest.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,12 @@ on:
1717
permissions:
1818
contents: write
1919

20+
# Serialize ingests: concurrent repository_dispatch deliveries race on
21+
# the commit+push and one writer loses. Queue them instead.
22+
concurrency:
23+
group: verisimdb-data-ingest
24+
cancel-in-progress: false
25+
2026
jobs:
2127
ingest:
2228
runs-on: ubuntu-latest

scripts/scan-all.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,9 @@ DEFAULT_REPOS=(
1616
)
1717

1818
REPOS_TO_SCAN=("${@:-${DEFAULT_REPOS[@]}}")
19-
if [[ -d "/var$REPOS_DIR" ]]; then
20-
REPOS_BASE="/var$REPOS_DIR"
19+
# REPOS_DIR (if set) must be an absolute path to the repo collection.
20+
if [[ -n "${REPOS_DIR:-}" && -d "${REPOS_DIR}" ]]; then
21+
REPOS_BASE="${REPOS_DIR}"
2122
else
2223
REPOS_BASE="$HOME/Documents/hyperpolymath-repos"
2324
fi

0 commit comments

Comments
 (0)