Skip to content

ci: cached setup and PgBouncer install, Docker build check on every PR - #36

Merged
BackStacked merged 2 commits into
mainfrom
claude/project-thread-dsh6g4
Nov 11, 2026
Merged

BackStacked merged 2 commits into
mainfrom
claude/project-thread-dsh6g4

Conversation

@BackStacked

Copy link
Copy Markdown
Contributor

Requested by Himanshu · project thread

What and why

Needs mgpl-diagnostics/.github#18 merged first (shared actions).

  • ci.yml: setup through mgpl-diagnostics/.github/actions/node (npm cache, MongoDB binary cache, npm ci). PgBouncer comes from actions/apt (cached .debs, retries, time limits): main's CI failed today after apt-get install pgbouncer hung 5.5 minutes on a mirror DNS outage.
  • Runs on main are no longer cancelled by the next merge (cancel-in-progress only for PRs), so every main commit keeps a CI result. The job id stays ci (required check).
  • New docker.yml: builds the Dockerfile (linux/amd64) on every PR and on main without pushing, with layer caching, then loads dist/index.js inside the image (every kit, report-kit's native resvg included, from the pruned node_modules). Skipped for docs/tests-only changes. Not a required check; release.yml is unchanged and still the only thing that pushes.

Type

  • Refactor / tooling

Checklist

  • CI green on this branch (timings in the thread)
  • No real patient data, contact details or signatures

Generated by Claude Code

…t caches), keep main runs

Temporary: points at the .github PR branch to measure; switched to @main before merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018jBohnZEym7gZYsPkBCmVg
@BackStacked BackStacked self-assigned this Nov 11, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018jBohnZEym7gZYsPkBCmVg

Copy link
Copy Markdown
Contributor Author

ci fails on purpose for now: Can't find 'action.yml' … for action 'mgpl-diagnostics/.github/actions/node@main'. This PR now uses the shared actions from main, and they arrive with mgpl-diagnostics/.github#18. The same workflow passed on the earlier commit, which used the branch ref. Merge .github#18 first, then re-run this check.


Generated by Claude Code

Copy link
Copy Markdown
Contributor Author

The shared action now loads. The re-run fails at install with No matching version found for @mgpl-diagnostics/client@^1.10.0. That requirement comes from main (#35), not from this PR. The client repo has 1.10.0 on main, but the version was never tagged or published (its latest tag is v1.9.0), so CI on main fails the same way. Publishing client 1.10.0 (tag v1.10.0) fixes both. Then re-run this check.


Generated by Claude Code

@BackStacked
BackStacked merged commit 58b1787 into main Nov 11, 2026
0 of 2 checks passed
@BackStacked
BackStacked deleted the claude/project-thread-dsh6g4 branch November 11, 2026 19:08
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.

2 participants