Extract Forgejo work-item + SCM operations behind a provider boundary (from #508 scope 4) #553

New issue

coilysiren · 2026-07-03T07:01:27Z

coilysiren commented

2026-07-03 07:01:27 +00:00

Follow-up from #508 (scope 4), split out so #508 could land its concrete slices (workflow modes + the docs reframe).

Goal

Progressively isolate Forgejo-specific work-item and SCM behavior behind provider-shaped seams, so future GitHub/GitLab support is architecturally plausible without rewriting the driver. Interfaces should evolve from the current code, not be imposed wholesale (per #508).

Why deferred from #508

This is a broad, cross-cutting refactor of the dispatch/reserve/comment/close and clone/push paths (cmd/ward/agent.go, forgejo_ops.go, git*.go, the reaper). #508 explicitly listed it as its own breakout; landing it alongside workflow modes would have made #508 unreviewably large. Note the GitHub lane (ward#489) already splits some landing behavior by forge, which is a natural starting seam.

Sketch (from #508, to evolve from real code)

WorkItemProvider: ParseRef, Read, ReadThread, Reserve, Release, Comment, Close.
SCMProvider: CloneURL, PushURL, OpenPR, ReadCI, Merge.

Acceptance (from #508)

Forgejo-specific parse/comment/list behavior progressively isolated behind provider-shaped functions/interfaces.
Existing Forgejo behavior unchanged.
New workflow-mode work (the #508 --workflow slice) does not deepen Forgejo coupling.
Public docs say "work items"/"issues" where generic, "Forgejo issues" only where implementation-specific.
Future GitHub/GitLab support plausible without rewriting the driver.

Follow-up from #508 (scope 4), split out so #508 could land its concrete slices (workflow modes + the docs reframe). ## Goal Progressively isolate Forgejo-specific work-item and SCM behavior behind provider-shaped seams, so future GitHub/GitLab support is architecturally plausible without rewriting the driver. Interfaces should **evolve from the current code**, not be imposed wholesale (per #508). ## Why deferred from #508 This is a broad, cross-cutting refactor of the dispatch/reserve/comment/close and clone/push paths (`cmd/ward/agent.go`, `forgejo_ops.go`, `git*.go`, the reaper). #508 explicitly listed it as its own breakout; landing it alongside workflow modes would have made #508 unreviewably large. Note the GitHub lane (ward#489) already splits some landing behavior by `forge`, which is a natural starting seam. ## Sketch (from #508, to evolve from real code) - `WorkItemProvider`: `ParseRef`, `Read`, `ReadThread`, `Reserve`, `Release`, `Comment`, `Close`. - `SCMProvider`: `CloneURL`, `PushURL`, `OpenPR`, `ReadCI`, `Merge`. ## Acceptance (from #508) - Forgejo-specific parse/comment/list behavior progressively isolated behind provider-shaped functions/interfaces. - Existing Forgejo behavior unchanged. - New workflow-mode work (the #508 `--workflow` slice) does not deepen Forgejo coupling. - Public docs say "work items"/"issues" where generic, "Forgejo issues" only where implementation-specific. - Future GitHub/GitLab support plausible without rewriting the driver.

coilysiren referenced this issue

2026-07-03 07:02:02 +00:00

feature request batch from codex convo #508

coilyco-ops added the

consult

P4

labels

2026-07-03 17:30:22 +00:00

coilysiren closed this issue

2026-07-04 10:36:59 +00:00

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

issue-1082

issue-1460

issue-1459-daemonless-alias

issue-1454

issue-1453

issue-1301

issue-1443

issue-1283

issue-1447

release

issue-1166

issue-1139

issue-1307

issue-1308

issue-1309

issue-1435

issue-1430

issue-1428

issue-1388

issue-1420

issue-1422

issue-1423

issue-1417

issue-1408

issue-1335

issue-1412

issue-1411

issue-1402

issue-1403

issue-1402-scoop-best-effort

issue-1336

issue-1334

issue-1393

issue-1338

issue-1397

issue-1392

issue-1339

issue-1340

issue-1300

issue-1288

issue-1304

issue-1270

issue-1381

issue-1244

issue-1227

issue-1380

issue-996

issue-1016

issue-1375

issue-1371

issue-1372

issue-1370

issue-1364

issue-1175

issue-1234

issue-1360

issue-1358

issue-1353

issue-1242

issue-1346

issue-1347

issue-1342

issue-1321

issue-1073-rebased

issue-1237-friction-events

issue-1320

issue-1344

issue-1073

issue-1275

issue-1332-replacement

issue-1296

issue-1330-replacement

issue-1224-rebased

issue-1323

issue-993

issue-1243

issue-871

issue-1298

issue-1315

issue-1312

issue-1314

issue-1313

issue-1310

issue-1282

issue-1224

issue-1173

issue-1000

issue-1007

issue-1281

issue-1011

issue-1006-recovery-note

issue-998

issue-1221

issue-1240

issue-1170

issue-1239

issue-1172

issue-1168

issue-1174

issue-1201

issue-1112

issue-1179

issue-1246

issue-1247

issue-1245

issue-1050

issue-1105

issue-1041

issue-1120

issue-1197

issue-1111

issue-1225

issue-1196

issue-1171

issue-1194

issue-1203

issue-1198

issue-1202

issue-1200

issue-1213

issue-1205

issue-1207

issue-1209

issue-1206

issue-1195

issue-1191

issue-1188

issue-1180

issue-1147

issue-1149

issue-1134

issue-1143

issue-1145

issue-1140

issue-1094

issue-1129

issue-1142

stash/agentic-os-image-default-rename

fix/test-env-isolation

docs/features-two-stage-release

fix/promote-server-scheme

fix/smartdefaults-error-source

feat/two-stage-release

issue-1086

fix/placeholder-sentinels

fix/migrate-off-cli-guard-dispatch

fix/test-yml-duplicate-main-gate

issue-1014-landing-fix

issue-911-pr

issue-1109

issue-1008

issue-999

issue-861

issue-1074

issue-1076

issue-1049

issue-1039

issue-994

issue-991

issue-1020

issue-1067

issue-913

issue-982

issue-1034

issue-1005

issue-1031

issue-990

issue-1062

issue-995

issue-1057

issue-905

issue-911

issue-1042

issue-1021

issue-1009

issue-1014

issue-1001

issue-978

issue-985

issue-950

issue-979

issue-977

issue-971

issue-935

issue-970

issue-969

issue-956

issue-965

issue-964

issue-962-restore-readme

issue-951

issue-958-container-config-guard

issue-704

issue-697

issue-896

issue-885

issue-876

issue-895

issue-893

issue-940

issue-923

issue-942

issue-952

issue-952-director

issue-936

issue-937

issue-931

issue-924

issue-920

issue-635

issue-903

issue-892

issue-874

issue-879

issue-890

issue-910

issue-875

issue-897

issue-894

issue-637

issue-842

issue-834

issue-860

issue-541

issue-723

issue-830-github-authoritative

issue-853

issue-634

issue-845

issue-846

issue-843

issue-496

issue-832

issue-777

issue-778

issue-836

issue-700

issue-695

issue-124-share-parser

issue-671

issue-722

issue-694

issue-824

issue-640

issue-735

issue-641

issue-642

issue-818

issue-812

issue-733-clean

issue-804

issue-805

issue-803

issue-735-remove-director-consult

issue-797

issue-801

issue-795

issue-791

issue-790

issue-787

issue-780

issue-737-defer-signoz

issue-732

issue-737-signoz-deferred

issue-734

issue-736

issue-744

issue-769

issue-767

issue-760

issue-763

ward-salvage/ward-c6aa5da9

ward-salvage/ward-94dd7346

issue-729

ward-salvage/ward-58aa3fe3

issue-733

ward-salvage/ward-ff6c509f

ward-salvage/ward-e80f0460

issue-737

ward-salvage/ward-3e2e4760

issue-731

ward-salvage/ward-649addd7

ward-salvage/ward-890e4d29

ward-salvage/ward-645b750b

ward-salvage/ward-bf851a72

ward-salvage/ward-98c8652f

ward-salvage/ward-bb10b620

issue-724

ward-salvage/ward-1ee9be1c

ward-salvage/ward-d18595f6

issue-124

ward-salvage/ward-5f914692

ward-salvage/ward-6ca05cbd

ward-salvage/ward-14f676cf

ward-salvage/ward-de811c20

ward-salvage/ward-eba3e824

ward-salvage/ward-16eb4ad0

ward-salvage/ward-c58e9c43

ward-salvage/ward-7487270b

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

coilyco-flight-deck/ward#553

Rows
Columns