Skip to content

fix(ci): expose failed upstream sync results - #65

Merged
pRizz merged 1 commit into
pRizz:mainfrom
thebitcoinnova:codex/upstream-sync-observability
Jun 7, 2026
Merged

fix(ci): expose failed upstream sync results#65
pRizz merged 1 commit into
pRizz:mainfrom
thebitcoinnova:codex/upstream-sync-observability

Conversation

@pRizz

@pRizz pRizz commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Print the structured upstream sync result to the Actions log when scripts/github-actions/upstream-sync.sh run-sync sees bun run sync:upstream --json exit nonzero.
  • Keep successful sync runs quiet and leave the existing summary/result output contract unchanged.
  • Add shell-wrapper coverage for both the failed-sync logging path and the quiet success path, and include it in bun run test:deploy.

Verification

  • bun test scripts/github-actions/upstream-sync-sh.test.ts scripts/github-actions/upstream-sync-summary.test.ts scripts/lib/upstream-sync-main.test.ts scripts/lib/upstream-sync.test.ts
  • bun run test:deploy
  • bun run biome:check
  • bun run studio:lint
  • bun run typecheck
  • bun run studio:typecheck
  • bun run --filter @openlinks/studio-api test
  • bun run studio:test:integration

Upstream PR hygiene

  • Branch rebuilt from upstream/main.
  • Fork-owned paths excluded from git diff --stat upstream/main...HEAD; the PR diff is limited to package.json and scripts/github-actions/*.

@pRizz
pRizz merged commit b5e8c15 into pRizz:main Jun 7, 2026
2 checks passed
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