Skip to content

Commit 23eb372

Browse files
ulemonsulemons
committed
fix: add resolution path
Signed-off-by: Umberto Sgueglia <usgueglia@contractor.linuxfoundation.org>
1 parent 689cf2a commit 23eb372

6 files changed

Lines changed: 66 additions & 13 deletions

File tree

‎backend/src/api/public/v1/packages/getPackage.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,8 @@ export async function getPackage(req: Request, res: Response): Promise<void> {
9393
status: (pkg.stewardshipStatus ?? 'unassigned') as StewardshipStatus,
9494
stewards: stewardshipSummary?.stewards ?? null,
9595
lastActivityAt: stewardshipSummary?.lastActivityAt ?? null,
96+
resolutionPath: pkg.stewardshipResolutionPath ?? null,
97+
statusNote: pkg.stewardshipStatusNote ?? null,
9698
},
9799
history: {},
98100
})

‎backend/src/api/public/v1/packages/openapi.yaml‎

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -85,14 +85,17 @@ components:
8585

8686
Steward:
8787
type: object
88-
required: [userId, name, role, assignedAt]
88+
required: [userId, role, assignedAt]
8989
properties:
9090
userId:
9191
type: string
92-
description: LFID.
93-
example: jdoe
92+
description: Auth0 sub of the assigned steward.
93+
example: abc123
9494
name:
95-
type: string
95+
type:
96+
- string
97+
- 'null'
98+
description: Display name of the steward. Null if not available.
9699
example: Jonathan R.
97100
role:
98101
type: string
@@ -418,12 +421,12 @@ components:
418421
- 'null'
419422
stewardship:
420423
type: object
421-
description: Stewardship state. In v1 always unassigned with no stewards or activity.
424+
description: Stewardship state.
422425
properties:
423426
status:
424427
$ref: '#/components/schemas/StewardshipStatus'
425428
stewards:
426-
description: Assigned stewards or null. Null in v1.
429+
description: Assigned stewards or null.
427430
oneOf:
428431
- type: array
429432
items:
@@ -434,7 +437,16 @@ components:
434437
- string
435438
- 'null'
436439
format: date-time
437-
description: Null in v1.
440+
resolutionPath:
441+
type:
442+
- string
443+
- 'null'
444+
description: Set on `escalated` status. Null for all other statuses.
445+
statusNote:
446+
type:
447+
- string
448+
- 'null'
449+
description: Free-text note for the current status.
438450
history:
439451
type: object
440452
description: Package history data. Empty in v1.

‎backend/src/api/public/v1/stewardships/openapi.yaml‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,17 @@ components:
9494
- stepped_down
9595
- no_longer_critical
9696
- 'null'
97+
resolutionPath:
98+
description: Set on `escalated` status. Null for all other statuses.
99+
oneOf:
100+
- $ref: '#/components/schemas/EscalationResolutionPath'
101+
- type: 'null'
102+
statusNote:
103+
type:
104+
- string
105+
- 'null'
106+
description: Free-text note for the current status. Set by escalate or updateStatus. Null on open.
107+
example: Contacted maintainer, no response after 30 days.
97108
createdAt:
98109
type: string
99110
format: date-time
@@ -115,6 +126,12 @@ components:
115126
type: string
116127
description: Auth0 sub of the assigned steward.
117128
example: abc123
129+
name:
130+
type:
131+
- string
132+
- 'null'
133+
description: Display name of the steward. Null if not available.
134+
example: Jonathan R.
118135
role:
119136
type: string
120137
enum: [lead, co_steward]

‎backend/src/osspckgs/migrations/V1781300000__stewardship-status-path-note.sql‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
ALTER TABLE stewardships
2+
ADD COLUMN IF NOT EXISTS resolution_path TEXT,
3+
ADD COLUMN IF NOT EXISTS status_note TEXT;

‎services/libs/data-access-layer/src/osspckgs/api.ts‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -276,6 +276,8 @@ export interface PackageDetailRow {
276276
stewardshipId: string | null
277277
stewardshipStatus: string | null
278278
stewardshipLastStatusAt: Date | null
279+
stewardshipResolutionPath: string | null
280+
stewardshipStatusNote: string | null
279281
// from package_repos + repos
280282
repoUrl: string | null
281283
repoMappingConfidence: number | null
@@ -319,6 +321,8 @@ export async function getPackageDetailByPurl(
319321
s.id::text AS "stewardshipId",
320322
s.status AS "stewardshipStatus",
321323
s.last_status_at AS "stewardshipLastStatusAt",
324+
s.resolution_path AS "stewardshipResolutionPath",
325+
s.status_note AS "stewardshipStatusNote",
322326
-- best repo link (highest confidence, prefer declared)
323327
r.url AS "repoUrl",
324328
pr.confidence AS "repoMappingConfidence",

‎services/libs/data-access-layer/src/osspckgs/stewardships.ts‎

Lines changed: 21 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@ export interface StewardshipRecord {
99
openedAt: string | null
1010
lastStatusAt: string | null
1111
inactiveReason: string | null
12+
resolutionPath: string | null
13+
statusNote: string | null
1214
createdAt: string
1315
updatedAt: string
1416
}
@@ -17,6 +19,7 @@ export interface StewardshipStewardRecord {
1719
id: string
1820
stewardshipId: string
1921
userId: string
22+
name: string | null
2023
role: string
2124
assignedAt: string
2225
assignedBy: string | null
@@ -89,6 +92,7 @@ function mapStewardStewardRow(row: Record<string, unknown>): StewardshipStewardR
8992
id: String(row.id),
9093
stewardshipId: String(row.stewardship_id),
9194
userId: String(row.user_id),
95+
name: null,
9296
role: String(row.role),
9397
assignedAt: toIso(row.assigned_at),
9498
assignedBy: row.assigned_by ? String(row.assigned_by) : null,
@@ -105,6 +109,8 @@ function mapStewardshipRow(row: Record<string, unknown>): StewardshipRecord {
105109
openedAt: row.opened_at ? toIso(row.opened_at) : null,
106110
lastStatusAt: row.last_status_at ? toIso(row.last_status_at) : null,
107111
inactiveReason: row.inactive_reason ? String(row.inactive_reason) : null,
112+
resolutionPath: row.resolution_path ? String(row.resolution_path) : null,
113+
statusNote: row.status_note ? String(row.status_note) : null,
108114
createdAt: toIso(row.created_at),
109115
updatedAt: toIso(row.updated_at),
110116
}
@@ -116,7 +122,7 @@ export async function getStewardshipById(
116122
): Promise<StewardshipRecord | null> {
117123
const row: Record<string, unknown> | null = await qx.selectOneOrNone(
118124
`SELECT id, package_id, status, origin, version, opened_at, last_status_at,
119-
inactive_reason, created_at, updated_at
125+
inactive_reason, resolution_path, status_note, created_at, updated_at
120126
FROM stewardships
121127
WHERE id = $(id)`,
122128
{ id },
@@ -151,12 +157,14 @@ export async function openStewardshipByPurl(
151157
FROM pkg
152158
ON CONFLICT (package_id) DO UPDATE
153159
SET status = 'open',
154-
opened_at = COALESCE(stewardships.opened_at, NOW()),
160+
opened_at = NOW(),
155161
last_status_at = NOW(),
156162
inactive_reason = NULL,
163+
resolution_path = NULL,
164+
status_note = NULL,
157165
updated_at = NOW()
158166
RETURNING id, package_id, status, origin, version, opened_at,
159-
last_status_at, inactive_reason, created_at, updated_at
167+
last_status_at, inactive_reason, resolution_path, status_note, created_at, updated_at
160168
),
161169
_log AS (
162170
INSERT INTO stewardship_activity (stewardship_id, actor_user_id, actor_type, activity_type, content)
@@ -288,10 +296,12 @@ export async function escalateStewardship(
288296
SET status = 'escalated',
289297
last_status_at = NOW(),
290298
inactive_reason = NULL,
299+
resolution_path = $(resolutionPath),
300+
status_note = $(statusNote),
291301
updated_at = NOW()
292302
WHERE id = $(stewardshipId)
293303
RETURNING id, package_id, status, origin, version, opened_at,
294-
last_status_at, inactive_reason, created_at, updated_at
304+
last_status_at, inactive_reason, resolution_path, status_note, created_at, updated_at
295305
),
296306
_log AS (
297307
INSERT INTO stewardship_activity
@@ -304,6 +314,8 @@ export async function escalateStewardship(
304314
`,
305315
{
306316
stewardshipId,
317+
resolutionPath: data.resolutionPath,
318+
statusNote: data.notes ?? null,
307319
actorUserId: data.actorUserId,
308320
content: `Escalated with resolution path: ${data.resolutionPath}`,
309321
metadata: JSON.stringify({
@@ -353,11 +365,13 @@ export async function updateStewardshipStatus(
353365
UPDATE stewardships
354366
SET status = $(status),
355367
last_status_at = NOW(),
356-
inactive_reason = CASE WHEN $(status) = 'inactive' THEN $(inactiveReason) ELSE NULL END,
368+
inactive_reason = CASE WHEN $(status) = 'inactive' THEN $(inactiveReason) ELSE inactive_reason END,
369+
resolution_path = NULL,
370+
status_note = $(statusNote),
357371
updated_at = NOW()
358372
WHERE id = $(stewardshipId)
359373
RETURNING id, package_id, status, origin, version, opened_at,
360-
last_status_at, inactive_reason, created_at, updated_at
374+
last_status_at, inactive_reason, resolution_path, status_note, created_at, updated_at
361375
),
362376
_log AS (
363377
INSERT INTO stewardship_activity
@@ -372,6 +386,7 @@ export async function updateStewardshipStatus(
372386
stewardshipId,
373387
status: data.status,
374388
inactiveReason: data.inactiveReason ?? null,
389+
statusNote: data.notes ?? null,
375390
actorUserId: data.actorUserId,
376391
content: `Status updated to ${data.status}`,
377392
metadata: JSON.stringify({

0 commit comments

Comments
 (0)