deps_add epic: title prefix check is case-sensitive - "epic: ..." gets double-prefixed to "Epic: epic: ..." #210

New issue

Closed

opened 2026-07-17 15:30:14 +00:00 by hexajon · 0 comments

hexajon commented

2026-07-17 15:30:14 +00:00

(Migrated from codeberg.org)

Fix: treat an existing "epic:" prefix case-insensitively (and tolerate surrounding whitespace) when deciding whether to prepend.

Acceptance criteria:

deps_add epic= on titles starting with "Epic: ", "epic: ", "EPIC: " leaves the title unchanged.
A title without the prefix still gains "Epic: " exactly once; running deps_add again stays idempotent.
Covered by a unit test alongside the existing epic-labelling tests.

Papercut from filing #206. deps_add with epic= prefixes the parent's title with "Epic: " unless it already has one, but the already-has-one check is case-sensitive: a title starting with "epic: " (lowercase) gets prefixed anyway, producing "Epic: epic: strict-omission destruction model - ...". Had to hand-edit the title back. Fix: treat an existing "epic:" prefix case-insensitively (and tolerate surrounding whitespace) when deciding whether to prepend. Acceptance criteria: - deps_add epic= on titles starting with "Epic: ", "epic: ", "EPIC: " leaves the title unchanged. - A title without the prefix still gains "Epic: " exactly once; running deps_add again stays idempotent. - Covered by a unit test alongside the existing epic-labelling tests.

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

sixfold-space/madtea#210

No description provided.

Rows
Columns