Skip to content

feat(workflow): return the content sha256 from run-scoped writes - #48

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-11903-return-the-content-sha256-in-corbitsartifacts-write-receipts
Oct 31, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-11903-return-the-content-sha256-in-corbitsartifacts-write-receipts

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11903

What changed

  • The run-scoped write routes (POST /artifacts, /artifacts/binary, /artifacts/link-file, PATCH /artifacts/:id) return contentSha256 with id and version, so artifact_write does too. The value is the hash stored with the version; the tenant routes already returned it.
  • CreatedWorkflowArtifact gains contentSha256: string | null (null only for versions written before hashes were stored).
  • README documents the field; for link-file the digest covers the stored preview text, not the workspace file.
  • artifact_write's description names the hash.

Verified

  • e2e covers create, binary create, link-file and revise; nulling any return site fails a test.
  • Sidecar tool output checked to carry the hash.
  • bun run check; test:e2e 308 pass; test:coverage 339 pass with the floor met (throwaway Postgres).

@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11903

@corbits-triage

corbits-triage Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve, request review.

@TheGreatAxios
TheGreatAxios merged commit 1360fe2 into main Oct 31, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant