jared/cc-os

Watch

Fork 0

Give "the valid tracker configured for a directory" one owner #1040

New issue

Closed

opened 2026-10-16 19:24:33 +00:00 by jared · 2 comments

jared commented

2026-10-16 19:24:33 +00:00

Owner

Problem

Three places answer "what tracker is configured here, and is it valid?", each a different way. A change to the lookup or validity rule must be made three times.

Observed

Evidence: lib/backlog/cli_tracker_requirement.rb:5-9 (Config::Reader + Tracker.valid?, exits via CliExit.fail_soft); lib/backlog/cli_command_wiring.rb:187-188 (dispatch_labels repeats the check, prints instead of exiting); lib/backlog/inspector/command.rb:23-27 (tracker_configured builds its own ConfigLocator + File.read + Config.new, no validity check). Rulings R20, R37, follow-ups 3 and 11.

Reproduce

n/a: duplicated lookup logic found in a crc-refactor tuning run; no runtime command reproduces it.

Acceptance criteria

One method returns the valid tracker for a directory or nil; CliTrackerRequirement calls it and keeps the exit; dispatch_labels calls it and keeps the print; Inspector::Command#tracker_configured calls it; the ConfigLocator/File.read copy is gone; existing tests pass.

Skeptic: VERDICT: CREATE — real duplication, recorded in rulings R20 and R37.

Origin

Trigger: crc-refactor tuning run on plugins/os-backlog, branch crc-os-backlog-2026-10-16 (ledger docs/crc/os-backlog/ledger.tsv, rulings docs/crc/os-backlog/rulings.md).
Improvised this session: none
Chain: duplicated tracker lookup in plugins/os-backlog ← rulings R20 and R37 (docs/crc/os-backlog/rulings.md) ← DESIGN
Root candidate: none beyond this ticket
Where: plugins/os-backlog/lib/backlog/cli_tracker_requirement.rb
Session: a5e278ab-89aa-4ed6-adb4-b70820788f0c
Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os--claude-worktrees-crc-os-backlog-2026-10-16/a5e278ab-89aa-4ed6-adb4-b70820788f0c.jsonl

## Problem Three places answer "what tracker is configured here, and is it valid?", each a different way. A change to the lookup or validity rule must be made three times. ## Observed Evidence: lib/backlog/cli_tracker_requirement.rb:5-9 (Config::Reader + Tracker.valid?, exits via CliExit.fail_soft); lib/backlog/cli_command_wiring.rb:187-188 (`dispatch_labels` repeats the check, prints instead of exiting); lib/backlog/inspector/command.rb:23-27 (`tracker_configured` builds its own ConfigLocator + File.read + Config.new, no validity check). Rulings R20, R37, follow-ups 3 and 11. ## Reproduce n/a: duplicated lookup logic found in a crc-refactor tuning run; no runtime command reproduces it. ## Acceptance criteria 1. One method returns the valid tracker for a directory or nil; CliTrackerRequirement calls it and keeps the exit; dispatch_labels calls it and keeps the print; Inspector::Command#tracker_configured calls it; the ConfigLocator/File.read copy is gone; existing tests pass. Skeptic: VERDICT: CREATE — real duplication, recorded in rulings R20 and R37. ## Origin - Trigger: crc-refactor tuning run on plugins/os-backlog, branch crc-os-backlog-2026-10-16 (ledger docs/crc/os-backlog/ledger.tsv, rulings docs/crc/os-backlog/rulings.md). - Improvised this session: none - Chain: duplicated tracker lookup in plugins/os-backlog ← rulings R20 and R37 (docs/crc/os-backlog/rulings.md) ← DESIGN - Root candidate: none beyond this ticket - Where: plugins/os-backlog/lib/backlog/cli_tracker_requirement.rb - Session: a5e278ab-89aa-4ed6-adb4-b70820788f0c - Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os--claude-worktrees-crc-os-backlog-2026-10-16/a5e278ab-89aa-4ed6-adb4-b70820788f0c.jsonl

jared added the

plugin/os-backlog

filed-by/agent

category/code-change

labels

2026-10-16 19:24:33 +00:00

jared referenced this issue

2026-10-16 19:26:00 +00:00

CRC wave 5: split Backlog::CliCommandHelpers #1025

jared referenced this issue

2026-10-16 19:26:05 +00:00

CRC wave 5: split Backlog::CliSupport #1027

jared referenced this issue

2026-10-16 19:26:27 +00:00

CRC wave 5: split Backlog::CliExit #1037

jared added

category/structure

and removed

category/code-change

labels

2026-10-18 19:31:58 +00:00

jared referenced this issue

2026-11-02 21:48:48 +00:00

os-sdlc cop Sdlc/Structural/PrecedenceInEdge: an edge picks which of two sources wins for one fact (A19 R1) #1509

jared referenced this issue

2026-11-06 20:06:25 +00:00

Add Project and Label, then delete the old issue backends #1621

jared commented

2026-11-06 20:06:36 +00:00

Author

Owner

Port slice #1621 does this work through Project. Close this ticket when #1621 ships.

jared commented

2026-11-07 12:22:23 +00:00

Author

Owner

Resolution

Done: Project is the one owner of the tracker lookup; Tracker::Resolver and the Tracker forwarders are deleted (#1621).

Evidence: main 1fb00d98, #1621

Follow-ups: none

## Resolution **Done:** Project is the one owner of the tracker lookup; Tracker::Resolver and the Tracker forwarders are deleted (#1621). **Evidence:** main 1fb00d98, #1621 **Follow-ups:** none

jared closed this issue

2026-11-07 12:22:23 +00:00

jared referenced this issue from a commit

2026-11-07 19:21:17 +00:00

feat(os-backlog): add Project and Label, delete Issues and the old backends (#1621)