Dashboard: the GitHub issue-number parser for a raw string exists three times #733

New issue

Open

opened 2026-10-07 01:33:40 +00:00 by claude · 3 comments

claude commented

2026-10-07 01:33:40 +00:00

Member

Found during IW-672 (IssueId.numericPart).

The GitHub issue-number parser on a raw String ("#N", "N", "TEAM-N") has three copies (dashboard/jvm/src/domain/IssueSource.scala:78, dashboard/jvm/src/application/IssueCacheService.scala:164, dashboard/jvm/src/application/IssueSearchService.scala:201); consider one owner.

Found during IW-672 (IssueId.numericPart). - [ ] The GitHub issue-number parser on a raw String ("#N", "N", "TEAM-N") has three copies (dashboard/jvm/src/domain/IssueSource.scala:78, dashboard/jvm/src/application/IssueCacheService.scala:164, dashboard/jvm/src/application/IssueSearchService.scala:201); consider one owner.

mph referenced this issue from a commit

2026-10-07 01:34:57 +00:00

docs(IW-672): review packet, release notes; file follow-ups

claude referenced this issue

2026-10-07 01:35:05 +00:00

IW-672: Add IssueId.numericPart instead of repeated split("-").last #735

claude commented

2026-10-08 00:56:34 +00:00

Author

Member

Intake: agent-ready
Workflow: waterfall
Reason: all gates passed
Analysis: project-management/issues/IW-733/analysis.md on branch IW-733

Intake: agent-ready Workflow: waterfall Reason: all gates passed Analysis: project-management/issues/IW-733/analysis.md on branch IW-733

claude added the

agent-ready

label

2026-10-08 00:56:34 +00:00

claude commented

2026-10-15 14:59:56 +00:00

Author

Member

[warn]  Using directives detected in multiple files:
- /home/mph/.local/share/iw/plugins/kanon/lib/deps.scala:1:1-7:41
It is recommended to keep them centralized in the /home/mph/Devel/iw/iw-cli-IW-733/core/project.scala file.

[error] /home/mph/.local/share/iw/plugins/kanon/lib/batch/ForgeCatchUp.scala:143:7
[error] Found:    Either[iw.core.model.TrackerError, Option[batch.ForgeState]]
[error] Required: Either[String, Option[batch.ForgeState]]
Compilation failed

Implement: needs-human Reason: batch-implement exit 1 — the kanon plugin did not compile against the core of this branch: lib/batch/ForgeCatchUp.scala:143 expects Either[String, …] from fetchPullRequestForBranch, and core now returns Either[TrackerError, …] (#778, #780). No phase ran, and no batch-implement.log was written. The plugin must follow the core change before this issue can run. ``` [warn] Using directives detected in multiple files: - /home/mph/.local/share/iw/plugins/kanon/lib/deps.scala:1:1-7:41 It is recommended to keep them centralized in the /home/mph/Devel/iw/iw-cli-IW-733/core/project.scala file. [error] /home/mph/.local/share/iw/plugins/kanon/lib/batch/ForgeCatchUp.scala:143:7 [error] Found: Either[iw.core.model.TrackerError, Option[batch.ForgeState]] [error] Required: Either[String, Option[batch.ForgeState]] Compilation failed ```

claude added

needs-human

and removed

agent-ready

labels

2026-10-15 14:59:56 +00:00

claude commented

2026-10-16 17:10:41 +00:00

Author

Member

Release: the implement loop failed this issue for a reason outside it. The kanon plugin did not compile against iw-cli main, because the tracker port now returns TrackerError (iw-cli #778). The plugin fix is dev-docs #484, and iw-cli 0.22.1 carries the core change. The issue goes back to the implement loop.

claude added

agent-ready

and removed

needs-human

labels

2026-10-16 17:10:41 +00:00