Skip to content

test(gcp-cloud-run): guard Docker workspace drift#2610

Merged
jrusso1020 merged 3 commits into
fix/2601-gcp-workspace-manifestsfrom
test/2601-gcp-dockerfile-regression
Jul 17, 2026
Merged

test(gcp-cloud-run): guard Docker workspace drift#2610
jrusso1020 merged 3 commits into
fix/2601-gcp-workspace-manifestsfrom
test/2601-gcp-dockerfile-regression

Conversation

@jrusso1020

Copy link
Copy Markdown
Collaborator

Stacked on #2608.

Summary

  • derive the Cloud Run runtime workspace closure from package manifests, including cyclic dependencies
  • validate matching manifest/source COPY destinations and full prerequisite builds before producer
  • add regression cases for missing or misdirected manifests/sources, missing or late builds, build subcommands, and producer/adapter ordering
  • wire the regression suite into the existing test:scripts CI entry point

Verification

  • bun run test:scripts (74 passed)
  • bunx oxlint packages/gcp-cloud-run/check-dockerfile-workspaces.mjs packages/gcp-cloud-run/check-dockerfile-workspaces.test.mjs
  • bunx fallow audit --base origin/main --fail-on-issues
  • git diff --check

@jrusso1020
jrusso1020 force-pushed the test/2601-gcp-dockerfile-regression branch from 15b03c4 to 754954a Compare July 17, 2026 04:03
@miguel-heygen
miguel-heygen force-pushed the fix/2601-gcp-workspace-manifests branch from 356fba0 to 5a2df8e Compare July 17, 2026 04:11
@jrusso1020
jrusso1020 force-pushed the test/2601-gcp-dockerfile-regression branch from 754954a to 51a70fc Compare July 17, 2026 04:15
@miguel-heygen
miguel-heygen force-pushed the fix/2601-gcp-workspace-manifests branch from 5a2df8e to 9ee94bb Compare July 17, 2026 04:27
@jrusso1020
jrusso1020 merged commit f2aceeb into fix/2601-gcp-workspace-manifests Jul 17, 2026
18 checks passed
@jrusso1020
jrusso1020 deleted the test/2601-gcp-dockerfile-regression branch July 17, 2026 04:31
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