Skip to content

feat: emit created_at on workflow_run_jobs - #1

Merged
malhussan merged 1 commit into
feature/fix-actions-streamsfrom
feature/workflow-job-created-at
Aug 13, 2026
Merged

feat: emit created_at on workflow_run_jobs#1
malhussan merged 1 commit into
feature/fix-actions-streamsfrom
feature/workflow-job-created-at

Conversation

@malhussan

@malhussan malhussan commented Aug 13, 2026

Copy link
Copy Markdown
Member

The GitHub API returns created_at on every workflow job, but the stream schema does not declare it, so the SDK drops it before it reaches the target.

This adds the property. Nothing else changes.

created_at is the time the job was queued. started_at is the time a runner picked it up.

Checked against one job, where the API returned created_at at 06:53:03 and started_at at 06:56:24.

Two notes:

  • This branches off feature/fix-actions-streams rather than main, because that branch carries the three workflow-stream fixes that are not on main: incremental loading for workflow_runs, the lookback window, and dropping state partitions for workflow_run_jobs. Basing this on main would mean losing those three when the pin moves.
  • Rows that are already loaded keep a null created_at. Only new syncs carry it, so a full re-sync is needed for history.

@malhussan
malhussan force-pushed the feature/workflow-job-created-at branch from d54e5cf to bf7487a Compare August 13, 2026 09:04
The API returns created_at on every job, but the stream schema does not declare
it, so the SDK drops it before it reaches the target.

created_at is the time the job was queued. started_at is the time a runner picked
it up.

@JohannesRudolph JohannesRudolph left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. probably the easiest way to get this in. I wonder however if we should also update the tap to its remote upstream - surely a lot has changed. OTOH - if it works it works, no point in rocking the boat

@malhussan
malhussan merged commit 9aa0e37 into feature/fix-actions-streams Aug 13, 2026
0 of 5 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.

2 participants