Skip to content

[codex] add server-owned job creation timestamps#3315

Open
TheGreatAion wants to merge 1 commit into
SecureBananaLabs:mainfrom
TheGreatAion:fix-3313-job-created-at
Open

[codex] add server-owned job creation timestamps#3315
TheGreatAion wants to merge 1 commit into
SecureBananaLabs:mainfrom
TheGreatAion:fix-3313-job-created-at

Conversation

@TheGreatAion
Copy link
Copy Markdown

Summary

  • add a server-owned ISO createdAt timestamp when jobs are created
  • keep caller-supplied timestamps from overriding the generated value
  • add focused service regression coverage for returned and stored timestamps

Why

The Prisma Job model defines createdAt, but the in-memory createJob service omitted it. This left newly created records without creation metadata and allowed a caller-supplied createdAt value to flow through the payload spread.

Demo

Watch the short job timestamp demo

Verification

  • node --test src/tests/health.test.js src/tests/job-created-at.test.js
  • git diff --check
  • Demo MP4 decoded successfully with ffmpeg -i job-created-at-demo.mp4 -f null -

The repository-wide npm test command still fails before discovery because node --test src/tests treats the test directory as a module path. That inherited harness issue is already tracked separately in #2014.

Closes #3313

/claim #743

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.

Job creation should include createdAt timestamps

1 participant