openova/openova
1
0
Fork 0
Code Issues Pull Requests Actions 4 Packages Projects Releases Wiki Activity

fix(sovereign-console): use DerivedJob.title not displayName/jobName (#669 follow-up) (#672)

Browse Source 
Build-ui failed in CI on `tsc -b` (which `tsc --noEmit` doesn't catch
locally without strict project-references). DerivedJob from
src/pages/sovereign/jobs.ts uses `title`, not the flat-Job
`displayName`/`jobName` fields. Use `dj.title || dj.id` for the
global-log component-name prefix.

Co-authored-by: hatiyildiz <hatice@openova.io>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
e3mrah 2026-05-03 15:10:33 +04:00 committed by GitHub
parent 3f1d5c106d
commit aae99cf9e0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
products/catalyst/bootstrap/ui/src/pages/sovereign/JobDetail.tsx
View File

@ -158,7 +158,11 @@ export function JobDetail({
const merged: Array<{ ts: number; line: LogLine }> = []
let counter = 1
for (const dj of derivedJobs) {
const label = dj.displayName ?? dj.jobName ?? dj.id
// DerivedJob (./jobs.ts) uses `title` for the human-readable
// label — not `displayName`/`jobName` which belong to the flat
// Job in @/lib/jobs.types. Fall back to the id when title is
// unset (it shouldn't be, but defensive).
const label = dj.title || dj.id
for (const ll of stepsToLogLines(dj.steps)) {
const ts = ll.timestamp ? Date.parse(ll.timestamp) : NaN
merged.push({