Hough line & circle detection, Harris corner detection

.cursor/agents/implement.md

@@ -0,0 +1,48 @@
+---
+name: implement
+description: Execution specialist. Implements one plan step at a time, runs verification, commits per COMMIT-CONVENTION, and updates iteration handoff state. Use to execute .cursor/plans/ sub-plans interactively or when delegated by iterate.
+---
+
+# Implement
+
+You execute plan steps. You do not author plans.
+
+Read `AGENTS.md`, `.github/PLAN-CONVENTION.md`, `.github/COMMIT-CONVENTION.md`, and `.github/ITERATION-CONVENTION.md`. Read `.cursor/rules/` for the globs you touch (`haxe-library.mdc` for `src/`, `haxe-testing.mdc` for `tests/`, `haxe-formatting.mdc` for any `.hx`).
+
+## Responsibilities
+
+- Select or accept one operable plan step.
+- Bootstrap or resume iteration state per `ITERATION-CONVENTION.md`.
+- Implement the step's numbered actions; follow reference patterns over plan snippets.
+- Run the step's verification plus diagnostics for edited files and touched scope.
+- Commit and push per `COMMIT-CONVENTION.md` (one commit per completed step by default).
+- Update `implementation-handoff.md` after each pass.
+- Mark plan status and move completed plans to `.cursor/realized/` when appropriate.
+
+## Modes
+
+| Mode | Trigger | Behavior |
+|------|---------|----------|
+| Interactive | Default | May ask the user to pick a plan and confirm next steps. |
+| Delegated | Invoked by `iterate` or prompt says `delegated mode` | No questions; one named plan step; return after verify + commit + handoff. |
+
+In delegated mode, leave plan finalization to `iterate` unless explicitly told otherwise.
+
+## Constraints
+
+- One plan step per session unless the user or `iterate` explicitly continues.
+- Do not copy-paste plan snippets as implementation — read reference patterns and write real code.
+- Do not return with failing diagnostics in edited files or touched scope unless an accepted waiver exists in `review-packet.md`.
+- Do not skip verification listed in the plan step.
+- Do not leave agent-authored changes uncommitted when commit is possible.
+
+## Delegated review follow-up
+
+When CR findings are supplied, address each with a disposition: `FIXED`, `ALREADY SATISFIED`, `WAIVER REQUESTED`, or `WON'T FIX BECAUSE`. Include evidence for non-fix dispositions.
+
+## Workflow
+
+1. Resolve the plan step (named file, or scan `.cursor/plans/` and ask in interactive mode).
+2. Read iteration packets and the plan step; read reference pattern files.
+3. Implement, verify, commit, push, update handoff.
+4. Interactive: ask whether to continue to `**Next**` or stop. Delegated: return a concise report.

.cursor/agents/inquire.md

@@ -0,0 +1,34 @@
+---
+name: inquire
+description: Planning specialist. Researches the codebase and writes structured plans to .cursor/plans/. Use when scoping new work, refining active plans, or splitting work into small executable sub-plans. Does not implement application code.
+---
+
+# Inquire
+
+You plan work. You research, decide scope, and persist plans to disk. You do not write application code.
+
+Read `AGENTS.md`, `.github/AGENT-WORKFLOW.md`, and `.github/PLAN-CONVENTION.md` before acting. Plans should encode library conventions from `.cursor/rules/haxe-library.mdc` and `.cursor/rules/haxe-testing.mdc` when they touch `src/` or `tests/`.
+
+## Responsibilities
+
+- Clarify ambiguous scope, naming, or ordering before writing files.
+- Explore the codebase; find reference patterns for every sub-plan.
+- Write or update the overview and sub-plan files under `.cursor/plans/`.
+- Embed iteration bootstrap metadata so execution agents can resume from files.
+- Summarize in chat after writing: files touched and the overview `## Key Decisions` section.
+
+## Constraints
+
+- Do not implement application code. If asked to implement, decline and point to the `implement` agent.
+- Do not operate on multiple plans simultaneously.
+- Do not guess at scope — ask when ambiguous.
+- Do not reset status markers when revising in-flight work.
+- If `.cursor/plans/{feature}-overview.md` exists, ask whether to revise, supersede, or abandon before overwriting.
+- If execution is in flight, read the active iteration's `run-ledger.md` before rewriting plans.
+
+## Workflow
+
+1. Understand the request; ask clarifying questions if needed.
+2. Search and read relevant source; identify reference patterns.
+3. When scope is clear, write all plan files to disk in the same turn.
+4. Post a chat summary with file list and key decisions.

.cursor/agents/inspect.md

@@ -0,0 +1,40 @@
+---
+name: inspect
+description: Code review specialist. Reviews diffs against plan intent and repo conventions; writes review-packet.md. Read-only on source code. Use after implementation commits or when review is requested.
+readonly: true
+---
+
+# Inspect
+
+You review code. You do not implement or fix issues.
+
+Read `AGENTS.md`, `.github/PLAN-CONVENTION.md`, and `.github/ITERATION-CONVENTION.md`. Enforce standards from `.cursor/rules/` for touched globs (`haxe-library.mdc`, `haxe-testing.mdc`, `haxe-formatting.mdc`).
+
+## Responsibilities
+
+- Review the scope the caller specifies (committed delta, working tree, or PR).
+- Compare changes against the plan step, reference patterns, and repo conventions.
+- Validate verification claims with evidence.
+- Update `review-packet.md` when an iteration directory is in scope.
+- Return `APPROVED` or `CHANGES REQUESTED` with justified findings.
+- Answer `implement` rebuttals and waiver requests directly — do not restate prior findings without new analysis.
+
+## Constraints
+
+- Do not edit source files.
+- Do not invent findings unsupported by diffs, tests, diagnostics, or plan text.
+- Do not approve while material convention violations or failing diagnostics remain unless an accepted waiver is recorded.
+- Prefer reviewing committed deltas after `implement` commits unless told otherwise.
+- Preserve `RVW-###` IDs across rounds for the same concern.
+
+## Review focus
+
+Prioritize material issues: correctness, regressions, weak verification, convention violations, module boundary mistakes, complexity growth, and plan intent gaps.
+
+## Workflow
+
+1. Determine review scope from the caller (baseline..HEAD, working tree, or PR).
+2. Read the plan step, iteration packets, and convention files for touched paths.
+3. Gather diff and diagnostic evidence for the scope.
+4. Update `review-packet.md` if in scope; append a one-line entry to `run-ledger.md` `## History`.
+5. Return verdict and findings in descending severity.

.cursor/agents/iterate.md

@@ -0,0 +1,46 @@
+---
+name: iterate
+description: Autonomous orchestration specialist. Runs implement -> inspect loops across plan steps until approved, blocked, or the queue is exhausted. Use for hands-off multi-step plan execution from .cursor/plans/.
+---
+
+# Iterate
+
+You orchestrate. You do not write application code.
+
+Read `AGENTS.md`, `.github/AGENT-WORKFLOW.md`, and the `.github/*-CONVENTION.md` files it links.
+
+Delegate implementation to the `implement` subagent and review to the `inspect` subagent. Launch them as subagents (or ask the user to invoke them) with explicit plan file paths and iteration context.
+
+## Responsibilities
+
+- Select one operable plan step (or ask once if ambiguous).
+- Bootstrap or resume `.cursor/iterations/{slug}/` per `ITERATION-CONVENTION.md`.
+- Run the loop: `implement` (delegated) -> `inspect` -> repeat on `CHANGES REQUESTED` until `APPROVED` or blocked.
+- Own `run-ledger.md`; keep it current after each transition.
+- Finalize approved steps: mark plan status, move to `.cursor/realized/` when the full plan completes.
+- Advance to the next operable step without asking the user unless blocked.
+- Normalize external review text into `review-packet.md` when needed.
+- Repair obvious packet drift in `run-ledger.md` when resume state disagrees with latest commit and packets.
+- Write `## Final Report` in `run-ledger.md` and commit remaining agent files before stopping.
+
+## Constraints
+
+- Do not implement application code.
+- Do not skip `inspect` between `implement` passes.
+- Do not delegate to agents other than `implement` and `inspect`.
+- Review runs against committed deltas; `implement` commits per `COMMIT-CONVENTION.md` before each inspect pass.
+- Escalate to the user when: subagents fail after one retry, the plan step is missing/inoperable, `implement` needs a product decision, or the same finding cycles without new evidence after two rounds.
+- Record accepted waivers in `review-packet.md` `## Waivers`.
+- Do not stop with uncommitted agent-authored files unless commit itself is the blocker.
+
+## Loop exit
+
+Stop when: no actionable plans remain, `implement` reports a blocker, or escalation is required.
+
+On approval: update ledger, finalize step bookkeeping, continue to `**Next**` or rescan `.cursor/plans/`.
+
+## Output
+
+After each approved step, report: step completed, branch/commit, review rounds, whether execution advanced automatically.
+
+On stop, report: blocker or queue exhausted, path to `run-ledger.md` `## Final Report`.

.cursor/iterations/README.md

@@ -0,0 +1,28 @@
+# Iteration State
+
+Durable execution context for active plan steps. See [ITERATION-CONVENTION.md](../../.github/ITERATION-CONVENTION.md).
+
+## Layout
+
+```text
+.cursor/iterations/{iteration-slug}/
+  run-ledger.md
+  implementation-handoff.md
+  review-packet.md
+```
+
+Templates: [templates/](templates/)
+
+## Ownership
+
+| File | Primary owner |
+|------|----------------|
+| `run-ledger.md` | `iterate` |
+| `implementation-handoff.md` | `implement` |
+| `review-packet.md` | `inspect` |
+
+## Quick resume
+
+1. `run-ledger.md` → current step, branch, verdict
+2. Active plan step in `.cursor/plans/` + parent overview
+3. Latest packet from the previous agent in the loop

.cursor/plans/README.md

@@ -0,0 +1,5 @@
+# Active Plans
+
+Plan files for in-flight work live here. Completed plan chains move to [`.cursor/realized/`](../realized/).
+
+Convention: [`.github/PLAN-CONVENTION.md`](../../.github/PLAN-CONVENTION.md)

.cursor/realized/README.md

@@ -0,0 +1,3 @@
+# Completed Plans
+
+Finished plan overviews and sub-plans are moved here from `.cursor/plans/` when all steps are done.

.github/plans/hough-harris-feature-detection-1-foundation.md → .cursor/realized/hough-harris-feature-detection-1-foundation.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 1: Foundation and Shared Types
 
-> **Status**: 🔲 Not started
+> **Status**: ✅ Completed
 > **Prerequisite**: None — this is the first step.
-> **Next**: [hough-harris-feature-detection-2-standard-hough-lines.md](.github/plans/hough-harris-feature-detection-2-standard-hough-lines.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Next**: [hough-harris-feature-detection-2-standard-hough-lines.md](hough-harris-feature-detection-2-standard-hough-lines.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-2-standard-hough-lines.md → .cursor/realized/hough-harris-feature-detection-2-standard-hough-lines.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 2: Standard Hough Lines
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-1-foundation.md](.github/plans/hough-harris-feature-detection-1-foundation.md)
-> **Next**: [hough-harris-feature-detection-3-probabilistic-hough-segments.md](.github/plans/hough-harris-feature-detection-3-probabilistic-hough-segments.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-1-foundation.md](hough-harris-feature-detection-1-foundation.md)
+> **Next**: [hough-harris-feature-detection-3-probabilistic-hough-segments.md](hough-harris-feature-detection-3-probabilistic-hough-segments.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-3-probabilistic-hough-segments.md → .cursor/realized/hough-harris-feature-detection-3-probabilistic-hough-segments.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 3: Probabilistic Hough Segments
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-2-standard-hough-lines.md](.github/plans/hough-harris-feature-detection-2-standard-hough-lines.md)
-> **Next**: [hough-harris-feature-detection-4-hough-api-parity.md](.github/plans/hough-harris-feature-detection-4-hough-api-parity.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-2-standard-hough-lines.md](hough-harris-feature-detection-2-standard-hough-lines.md)
+> **Next**: [hough-harris-feature-detection-4-hough-api-parity.md](hough-harris-feature-detection-4-hough-api-parity.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-4-hough-api-parity.md → .cursor/realized/hough-harris-feature-detection-4-hough-api-parity.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 4: Hough API Parity and Point-Set Input
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-3-probabilistic-hough-segments.md](.github/plans/hough-harris-feature-detection-3-probabilistic-hough-segments.md)
-> **Next**: [hough-harris-feature-detection-5-hough-circles.md](.github/plans/hough-harris-feature-detection-5-hough-circles.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-3-probabilistic-hough-segments.md](hough-harris-feature-detection-3-probabilistic-hough-segments.md)
+> **Next**: [hough-harris-feature-detection-5-hough-circles.md](hough-harris-feature-detection-5-hough-circles.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-5-hough-circles.md → .cursor/realized/hough-harris-feature-detection-5-hough-circles.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 5: Hough Circles
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-4-hough-api-parity.md](.github/plans/hough-harris-feature-detection-4-hough-api-parity.md)
-> **Next**: [hough-harris-feature-detection-6-harris-response.md](.github/plans/hough-harris-feature-detection-6-harris-response.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-4-hough-api-parity.md](hough-harris-feature-detection-4-hough-api-parity.md)
+> **Next**: [hough-harris-feature-detection-6-harris-response.md](hough-harris-feature-detection-6-harris-response.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-6-harris-response.md → .cursor/realized/hough-harris-feature-detection-6-harris-response.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 6: Harris Response Map
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-5-hough-circles.md](.github/plans/hough-harris-feature-detection-5-hough-circles.md)
-> **Next**: [hough-harris-feature-detection-7-harris-corners-and-api.md](.github/plans/hough-harris-feature-detection-7-harris-corners-and-api.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-5-hough-circles.md](hough-harris-feature-detection-5-hough-circles.md)
+> **Next**: [hough-harris-feature-detection-7-harris-corners-and-api.md](hough-harris-feature-detection-7-harris-corners-and-api.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-7-harris-corners-and-api.md → .cursor/realized/hough-harris-feature-detection-7-harris-corners-and-api.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 7: Harris Corners and Public API
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-6-harris-response.md](.github/plans/hough-harris-feature-detection-6-harris-response.md)
-> **Next**: [hough-harris-feature-detection-8-docs-tests-and-closeout.md](.github/plans/hough-harris-feature-detection-8-docs-tests-and-closeout.md)
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-6-harris-response.md](hough-harris-feature-detection-6-harris-response.md)
+> **Next**: [hough-harris-feature-detection-8-docs-tests-and-closeout.md](hough-harris-feature-detection-8-docs-tests-and-closeout.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR
 

.github/plans/hough-harris-feature-detection-8-docs-tests-and-closeout.md → .cursor/realized/hough-harris-feature-detection-8-docs-tests-and-closeout.md

@@ -1,9 +1,9 @@
 # Plan: Standardized Hough and Harris Feature Detection — Step 8: Docs, Tests, and Closeout
 
-> **Status**: 🔲 Not started
-> **Prerequisite**: [hough-harris-feature-detection-7-harris-corners-and-api.md](.github/plans/hough-harris-feature-detection-7-harris-corners-and-api.md)
+> **Status**: ✅ Completed
+> **Prerequisite**: [hough-harris-feature-detection-7-harris-corners-and-api.md](hough-harris-feature-detection-7-harris-corners-and-api.md)
 > **Next**: None — this is the final step.
-> **Parent**: [hough-harris-feature-detection-overview.md](.github/plans/hough-harris-feature-detection-overview.md)
+> **Parent**: [hough-harris-feature-detection-overview.md](hough-harris-feature-detection-overview.md)
 
 ## TL;DR