Skip to content

Fix infra dashboard history order#5172

Merged
yonromai merged 1 commit intomainfrom
codex/infra-dashboard-history-order
Apr 25, 2026
Merged

Fix infra dashboard history order#5172
yonromai merged 1 commit intomainfrom
codex/infra-dashboard-history-order

Conversation

@yonromai
Copy link
Copy Markdown
Contributor

Summary

Render the infra status page history strips oldest-to-newest so the most recent GitHub Build and Ferry status squares appear on the right.

Root Cause

The GitHub APIs return workflow runs and commit history newest-first. The dashboard used those arrays directly for the compact status strips, so time flowed right-to-left even though the latest-line logic correctly treated index 0 as newest.

Changes

  • Reverse the rendered Build strip while keeping commits[0] as the latest commit.
  • Reverse the rendered Ferry strip using original indices so slow-run detection and baseline math still use the newest-first source order.

Validation

  • npm run lint
  • npm run typecheck
  • npm run build

@yonromai yonromai added the agent-generated Created by automation/agent label Apr 25, 2026 — with ChatGPT Codex Connector
Copy link
Copy Markdown
Contributor Author

🤖 Screenshot check with deterministic mocked API data. The mock makes the latest Ferry run and latest Build commit amber, so the ordering is visible at a glance.

Before (origin/main): latest amber square is on the left.

Before: origin/main shows latest on left

After (this PR): latest amber square is on the right.

After: PR shows latest on right

Capture setup: local Vite dashboard, /api/* served by a mock server, viewport 1280x1200.

@yonromai yonromai changed the title [codex] Fix infra dashboard history order Fix infra dashboard history order Apr 25, 2026
@yonromai yonromai marked this pull request as ready for review April 25, 2026 02:30
@yonromai yonromai requested a review from ravwojdyla April 25, 2026 02:30
@yonromai yonromai enabled auto-merge (squash) April 25, 2026 02:31
Copy link
Copy Markdown
Contributor

@ravwojdyla ravwojdyla left a comment

Choose a reason for hiding this comment

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

:shipit:

@yonromai yonromai merged commit fc78e0c into main Apr 25, 2026
37 checks passed
@yonromai yonromai deleted the codex/infra-dashboard-history-order branch April 25, 2026 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-generated Created by automation/agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants