Skip to content

id: split pkgs/id/src/web/markdown.rs (5,284 production lines) — remove the F11.10 line-guard grandfather entry #4704

Open

Description

@dezren39

Why

pkgs/id/src/web/markdown.rs was measured at 5,284 production lines / 10,045 total on origin/main 0e26fa4b3 (triage task 2577 for #1411) — more than double the F11.10 2,500-production-line limit (#808). On current main c0e9d059a, the guard counts 5,350 production lines.

The regression guard added for #1411 grandfathers this one file explicitly so the check can pass while failing on any new violator. The guard is now on main via PR #4709. This issue is the linked removal condition: when markdown.rs is split below the limit, delete its grandfather entry in the same PR.

Scope

  • Split pkgs/id/src/web/markdown.rs into cohesive modules (codec/import/export seams are the natural fault lines) so no resulting production file exceeds 2,500 production lines.
  • No behavior change: the existing markdown test suites must pass unchanged.
  • Remove the grandfather entry for markdown.rs from .github/scripts/check-production-line-limit.sh in the same PR.

Acceptance criteria

Filed by repo queue worker (task 6592) as the split packet routed from #1411.

Activity

  1. added
    app/idPackage/app: id (pkgs/ or apps/ in this repo)
    on Nov 3, 2026
  2. dezren39 commented on Nov 3, 2026

    @dezren39
    MemberAuthor

    agent-note:v1 — state: handed off

    Queue task #6691 is handed to OpenCode for execution.

    • Session: ses_e5e60e1b0e3ffed4E104kZPRbU
    • Lane: opencode
    • Worker claim: repo-worker-code
    • Task: Split pkgs/id/src/web/markdown.rs and remove its production-line-limit grandfather entry; scope and acceptance are in task #6691.
  3. dezren39 commented on Nov 3, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · picked up

    • Agent: queue worker (repo-worker-code), OpenCode execution lane
    • Session / ID: task 6691 · session ses_e5e60e1b0e3ffed4E104kZPRbU
    • Found via: bound queue claim
    • Task: Split pkgs/id/src/web/markdown.rs and remove its grandfather entry from the production-line guard without behavior changes.
    • Expectation: effort m; cohesive modules below 2,500 production lines, markdown tests and guard pass.
    • Links: branch/worktree pending creation.
  4. added
    queue/blockedQueue or user escalation state
    status/blockedQueue or user escalation state
    and removed
    queue/claimedQueue worker lifecycle state
    on Nov 3, 2026
  5. dezren39 commented on Nov 3, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · blocked

  6. added
    queue/claimedQueue worker lifecycle state
    and removed
    status/blockedQueue or user escalation state
    queue/blockedQueue or user escalation state
    on Nov 3, 2026
  7. 53 remaining items

  8. added
    status/parkedDeliberately paused with a recorded resume condition
    on Nov 4, 2026
  9. dezren39 commented on Nov 4, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · picked up

  10. dezren39 commented on Nov 4, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · parked

  11. dezren39 commented on Nov 4, 2026

    @dezren39
    MemberAuthor
    \n**Agent note — picked up, revalidated, parked for a full implementation window (task 6691)**\n- Task: task 6691 (repo-health-code-pilot, lane 1a180ae5-384f-4e71-8d77-08f7b1652c75, 2026-11-04). Pickup and park notes combined in this comment.\n- Fresh GitHub state: issue #4704 is OPEN, assigned to dezren39, with no hard-stop labels. PR #4709 is MERGED; fresh open-PR search for #4704 returned none.\n- Local state: expected worktree /home/user/git/fleet/task-6691 does not exist; neither local nor remote branch queue/6691-4704-split-markdown was present. No source changes were started and the primary clone was left untouched.\n\n**Park / resume**\n- Parked because this scheduled lane has about 30 minutes while the behavior-preserving split is estimated at 120 minutes; no implementation should begin in a window likely to end mid-refactor.\n- Resume after the next full implementation window's fresh revalidation. Create the task worktree/branch then, split only markdown.rs, remove only its guard entry, preserve #4715/#4716/#4717 entries, and run the packet's markdown tests, production-line guard, and git diff --check.\n- Issue labels are queue/parked, status/parked, work/assigned.\n
  12. added
    queue/claimedQueue worker lifecycle state
    and removed
    queue/parkedQueue task is parked (waiting on CI, time, or an event)
    status/parkedDeliberately paused with a recorded resume condition
    on Nov 4, 2026
  13. dezren39 commented on Nov 4, 2026

    @dezren39
    MemberAuthor

    agent-note:v1
    event: picked_up
    task: 6691 (small_fix, repo-health-code-pilot; repo-worker-code; OpenChamber scheduled lane 9aca43b4-5439-4939-9f71-323f5d32521b)
    session: ses_e5e5747e1acffeD0WKy90aD3dW
    state: claimed and freshly revalidated. Issue #4704 is OPEN, assigned to dezren39, with no hard-stop labels. Linked PR #4709 is MERGED; fresh open-PR search for #4704 found none.
    labels: queue/claimed, status/in-progress, work/assigned.
    next: inspect task branch/worktree and laptop main freshness; decide whether the remaining scope can safely reach a terminal state within this scheduled run's 30-minute cap. No source edits yet.

  14. added
    queue/queuedHas a queued task in the Supabase agent queue
    and removed
    queue/claimedQueue worker lifecycle state
    work/assignedQueue worker lifecycle state
    on Nov 5, 2026
  15. dezren39 commented on Nov 5, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · handed off

    • Agent: fleet monitor
    • Session / ID: task 6691 (repo-fleet-monitor run 2026-11-05T15:45:47Z)
    • Found via: monitor reconciliation (Duty 1 claim/label divergence repair)
    • Task: Claim evaporated: task 6691 is queued and unclaimed in repo-health-code-pilot, but this issue still carried claimed labels. Labels reset to the truth (queue/queued + status/needs-triage). Checkpoint stage at evaporation: handed_off.
    • Expectation: The next lane claim picks task 6691 up from the queued state and posts a fresh pickup note.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

app/idPackage/app: id (pkgs/ or apps/ in this repo)queue/queuedHas a queued task in the Supabase agent queuesource/creatorFiled by the new-issue-creator jobstatus/needs-triageNot yet triaged; needs a first looktask/triageQueue packet kind: verify, reproduce, and route this item

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions