Skip to content

fix: qualify Temporal with its declared persistence dependency - #21

Merged
antoinetoussaint-byte merged 4 commits into
mainfrom
fix/conformance-dependency-fixture
Sep 22, 2026
Merged

antoinetoussaint-byte merged 4 commits into
mainfrom
fix/conformance-dependency-fixture

Conversation

@antoinetoussaint-byte

@antoinetoussaint-byte antoinetoussaint-byte commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Fixes at the Temporal owner

  • Use the real multi-service fixture and published Postgres v0.0.134; the generated single-service fixture could not satisfy persistence.
  • Advertise a readiness test with START_STACK and check the running Temporal frontend with GetSystemInfo. Invalid test selections and an unstarted stack fail.
  • Implement non-mutating sync for this service with no generated source and return the exact digest-pinned managed image from Builder.Build. Native lint/compile remain explicitly unsupported because there is no native service source.
  • Select the native Postgres configuration through Core before resolving credentials. Previously configuration ordering could send the embedded host process to host.docker.internal. Stop advertising an unimplemented container runtime; deployment image rendering remains supported.

Verification

Full owner suite, build, vet and all GitHub checks pass. Real agent CI with CLI #780 now passes all six stages (manifest, source, native/Linux build, audit, full fixture conformance, drift), zero failed or skipped stages. Readiness runs against the actual Postgres/Temporal stack. Regressions cover runtime projection order and container-only failure, loaded-service requirements, unchanged sync files, exact build image, test contract and invalid requests. Docker dependency audit retains GO-2026-4883 and GO-2026-4887, which have no upstream fix.

These are owner fixes, not CLI exceptions. No cloud deployment was performed. Release publication and downloaded-artifact verification follow separately in codefly-dev/core#622.

Refs codefly-dev/cli#779 and codefly-dev/cli#780.

The default one-service conformance workspace cannot run Temporal without Postgres. Use the owner existing full-stack fixture, select the candidate locally, and pin the published database dependency. CLI #779 supplies isolated dependency installation.
@antoinetoussaint-byte

Copy link
Copy Markdown
Contributor Author

Actual candidate conformance proved CLI fixture installation and lint/compile dependency setup, then failed at Runtime.Test because the agent had no test dependency declaration. The owner now advertises a default readiness suite with START_STACK, performs a real GetSystemInfo RPC, reports one passed/failed test, and rejects unsupported selections instead of returning empty success. Only implemented audit/SBOM operations are otherwise advertised; the prior inherited no-op lint/compile/build handlers are no longer presented as successful validation. Full-stack Go tests exercise the new RPC against real Temporal/Postgres and passed (21.613s); build/vet passed. Actual agent CI rerunning. No release claimed yet.

@antoinetoussaint-byte
antoinetoussaint-byte marked this pull request as ready for review September 22, 2026 06:06
@antoinetoussaint-byte
antoinetoussaint-byte merged commit 87f62f7 into main Sep 22, 2026
3 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