3v5hs37o3u58u98r1cf738t47u3scoq8 · 168 rows
| id | issue_id | event_type | actor | old_value | new_value | comment | created_at |
|---|---|---|---|---|---|---|---|
| 019f73e8-6c27-7090-b949-807aa368c32e | ah-6ei | closed | Eugene Blikh | Verified live on project 2 / agent-1. Flushed out the real gap: (1) live config lacked design/review task_types+designer/validator roles+skills_dir; (2) NO CI deploy lane existed (.build.yml) and the agent1-deploy key was lost. Built the CI lane (.build.yml, commit 38dd035, build #226 SUCCESS: builds 3 cmds + ships bin/prompts/skills to /opt/agent-hub + restarts), regenerated+registered the deploy key (builds secret cc6bd011), wired designer/validator into the live config. Then ran the flow: task 2 type:design -> designer -> In Review with ## Design (ae7db66); relabel type:review+Ready -> validator -> ## Review changes-requested on the SAME file/branch (374b583, agent/task-2). Validator even caught a real arithmetic bug in the design's edge-case table. | NULL | 2026-07-18T09:27:19Z | |
| 019f73e8-6f29-7a21-8d89-d6c6ecdd6af9 | ah-943 | created | Eugene Blikh | NULL | 2026-07-18T09:27:20Z | ||
| 019f73eb-2c60-7b2b-bbf9-bf0ec606ee9a | ah-bkr | created | Eugene Blikh | NULL | 2026-07-18T09:30:20Z | ||
| 019f73eb-2ea6-7991-8823-ac8492ca4fab | ah-tz0 | created | Eugene Blikh | NULL | 2026-07-18T09:30:20Z | ||
| 019f73eb-3109-774b-b077-31774b346f29 | ah-4el | created | Eugene Blikh | NULL | 2026-07-18T09:30:21Z | ||
| 019f73eb-336f-792f-b2ff-2893e3883189 | ah-07g | created | Eugene Blikh | NULL | 2026-07-18T09:30:21Z | ||
| 019f73f2-e8c0-7de2-a29d-cf94bf14802a | ah-bkr | claimed | Eugene Blikh | {"id":"ah-bkr","title":"Vikunja comments render literal markdown fences — post HTML instead","description":"The In Review notification comment is built as markdown (internal/reconcile/comments.go successComment: triple-backtick fences around the diff stat, backtick spans). Vikunja stores/renders task comments as HTML (TipTap rich-text), so the markdown shows LITERALLY in the UI (screenshot: raw fences and + diffstat shown verbatim). internal/vikunja Comment() sends the string as-is to PUT /tasks/{id}/comments. Fix: render markdown to HTML (or emit HTML) before posting. Affects every daemon comment.","status":"open","priority":2,"issue_type":"bug","owner":"bigbes@gmail.com","created_at":"2026-07-18T06:30:20Z","created_by":"Eugene Blikh","updated_at":"2026-07-18T06:30:20Z"} | {"assignee":"Eugene Blikh","status":"in_progress"} | NULL | 2026-07-18T09:38:46Z |
| 019f73f2-ea09-7eb5-a4f5-2d758685b21c | ah-tz0 | claimed | Eugene Blikh | {"id":"ah-tz0","title":"Surface agent closing summary / review verdict in the In Review comment","description":"domain.RunSummary carries only DiffStat + Commits, so the comment shows diffstat+commits+link but not what was done. Designer and validator both END with a one-line summary/verdict; that line is lost (only in docs/tasks/slug.md). Capture the agent final summary (from exit.json or last event) into RunSummary and render it in successComment. Prereq for verdict-driven automation.","status":"open","priority":3,"issue_type":"feature","owner":"bigbes@gmail.com","created_at":"2026-07-18T06:30:21Z","created_by":"Eugene Blikh","updated_at":"2026-07-18T06:30:21Z"} | {"assignee":"Eugene Blikh","status":"in_progress"} | NULL | 2026-07-18T09:38:47Z |
| 019f73f4-13b9-7e67-8c7d-c07cea00686a | ah-943 | claimed | Eugene Blikh | {"id":"ah-943","title":"Refresh phoebe-lab agent-hub provisioning template + bootstrap for designer/validator","description":"The live /etc/agent-hub/agenthub.yaml on agent-1 was hand-wired for project 2, repo agent-demo, deepseek model, AND now designer/validator roles + design/review task_types + skills_dir=/opt/agent-hub/skills. The phoebe-lab/agent-hub/config/agenthub.yaml.example (first-boot seed) is stale vs this: still project_id 1, repo demo, coder-only, no skills_dir/task_types. bootstrap.sh also only pre-creates /opt/agent-hub/{bin,prompts} (skills/ gets created by the CI tar overlay, which works but isn't declared). Update the example + bootstrap so a fresh box provisions the full role set. Also: keys/agent1-deploy.pub was regenerated this session (new ed25519, matching builds.sr.ht secret cc6bd011) and is modified-uncommitted in the phoebe-lab repo — commit it or a redeploy/bootstrap reinstalls the OLD dead key.","status":"open","priority":3,"issue_type":"task","owner":"bigbes@gmail.com","created_at":"2026-07-18T06:27:20Z","created_by":"Eugene Blikh","updated_at":"2026-07-18T06:27:20Z"} | {"assignee":"Eugene Blikh","status":"in_progress"} | NULL | 2026-07-18T09:40:03Z |
| 019f73f5-42a8-79a8-93fe-729d9069431d | ah-943 | closed | Eugene Blikh | Provisioning template (config/agenthub.yaml.example) now scaffolds skills_dir + designer/validator roles + design/review task_types; bootstrap.sh pre-creates /opt/agent-hub/skills. Committed phoebe-lab 34bcec3 (local; deploy via labng). Pubkey rotation committed earlier (abab805). | NULL | 2026-07-18T09:41:21Z | |
| 019f73f5-f38e-7bb4-91d4-89df1389eeb2 | ah-07g | updated | Eugene Blikh | {"id":"ah-07g","title":"Investigate unbounded .task/events.jsonl (202MB for one design run)","description":"task-2/.task/events.jsonl was 202 MB for a single ~2min run producing a 100-line doc. pi event stream persisted verbatim, no truncation/rotation, under work_root. work_root fills fast. Investigate pi verbosity, whether events.jsonl is needed post-run, add rotation/cap or prune-on-Done.","status":"open","priority":3,"issue_type":"task","owner":"bigbes@gmail.com","created_at":"2026-07-18T06:30:22Z","created_by":"Eugene Blikh","updated_at":"2026-07-18T06:30:22Z"} | {"notes":"INVESTIGATION (2026-07-18): events.jsonl is pi's raw event stream, persisted verbatim by pi in \u003cworktree\u003e/.task/. 210MB for task-2's ~2min run; lines up to ~100KB each (full message/context per event). Worktrees are NOT pruned on terminal state (task-1 and task-2 both still on disk under work_root). Two levers: (a) pi-side — reduce event verbosity / don't persist full context per event (pi is external, needs a flag or upstream change); (b) daemon-side — prune the worktree or at least .task/events.jsonl when a task reaches Done (NOT In Review, where zellij-attach/review still want it). Retention policy is a user call: keep-for-debug vs disk. No code changed yet — needs a retention-policy decision."} | NULL | 2026-07-18T09:42:06Z |
| 019f73f7-dca4-7059-8725-f625eb84b6b9 | ah-bkr | closed | Eugene Blikh | markdownToHTML converts outbound comments to HTML at the vikunja Comment() boundary (dep-free, escaping + placeholder-safe). Builders still emit markdown (comments_test green). Verified: 89 subtests pass; live Vikunja round-trip confirmed the API stores <p>/<pre><code>/<ul>/<a> intact (markdown showed literally before). Merged to master (f6098a9). | NULL | 2026-07-18T09:44:11Z | |
| 019f73f9-9709-7e63-a77b-bdabe403a5d8 | ah-tz0 | closed | Eugene Blikh | RunSummary gains a Summary field read from .task/summary.md (runner.readSummaryFile, trim+2000-rune tail-truncate, missing=empty). successComment renders it as a blockquote below the header. designer/validator/coder prompts now write their closing summary/verdict there. Verified: unit tests green in isolation AND combined with ah-bkr on master; full-module build OK. Merged (rebased onto master). NOTE: follow-up needed — the blockquote markdown isn't yet handled by ah-bkr's HTML converter. | NULL | 2026-07-18T09:46:04Z | |
| 019f73fb-8754-700f-bab0-e19243081740 | ah-0iz | created | Eugene Blikh | NULL | 2026-07-18T09:48:11Z | ||
| 019f73fb-88fe-73a9-b111-47216600e38a | ah-0iz | closed | Eugene Blikh | Added <blockquote> handling to markdownToHTML (mirrors the bullet-run logic; bare > = empty line). Unit tests + live Vikunja round-trip confirm blockquote+strong render. Committed on master. | NULL | 2026-07-18T09:48:12Z | |
| 019f7401-a9ef-7ae1-b259-5fa7af5a19ec | ah-a6f | created | Eugene Blikh | NULL | 2026-07-18T09:54:53Z | ||
| 019f7401-ab94-7a35-b6c2-eb65859d4b02 | ah-a6f | closed | Eugene Blikh | Updated 5 e2e comment assertions (branch/worktree/outcome) from markdown backticks to <code> HTML. Full e2e suite passes (10.95s). Root cause: ah-bkr subagent scoped to its own package; build-tagged e2e wasn't exercised until the integration sweep. | NULL | 2026-07-18T09:54:54Z | |
| 019f7402-35c5-7995-865f-81dc2d724d25 | ah-4el | updated | Eugene Blikh | {"id":"ah-4el","title":"Review-verdict-driven transition: changes-requested back to Ready for coder","description":"Every successful run lands In Review; no verdict-aware transition, so a validator changes-requested just sits for a human to drag back. Intentional for now (ah-gs7 human-driven handoff) but user asked for it. Build opt-in loop: parse validator verdict (needs machine-readable verdict), on changes-requested move card back to Ready under coder/fix role with the review as context. Ties into Stage 4 (ah-0ge).","status":"open","priority":3,"issue_type":"feature","owner":"bigbes@gmail.com","created_at":"2026-07-18T06:30:21Z","created_by":"Eugene Blikh","updated_at":"2026-07-18T06:30:21Z"} | {"notes":"DESIGN PROPOSAL (held for user sign-off — changes the core reconcile state machine; not live-verifiable while pushes/deploy are blocked).\n\nNow unblocked by ah-tz0 (.task/summary.md exists). Proposed:\n1. MACHINE-READABLE VERDICT: don't parse free-text. Have the validator write a structured line the daemon can key on — e.g. a first line 'Verdict: pass' | 'Verdict: changes-requested' in .task/summary.md (update prompts/validator.md + validate skill), and add a domain field Verdict (parsed in runner.Summary, empty for non-validator roles).\n2. OPT-IN CONFIG: task_types.review gains 'on_changes_requested: \u003cbucket|role\u003e' (default: none = current behavior, lands In Review for a human). When set e.g. to role 'coder', a changes-requested validator run moves the card to Ready under that role instead of In Review, carrying the review as context (the ## Review is already on the branch).\n3. LOOP GUARD: reuse the delegation generation cap (max_generation) or add a review_rounds cap so design-\u003ereview-\u003ecoder-\u003ereview can't cycle forever. A 'pass' verdict always lands In Review (human ships it).\n4. STATE MACHINE: this is a new transition in internal/reconcile (finalize path) + internal/domain/transitions.go. Cover with reconcile_test.go (injected clock/fakes) AND an e2e flow before shipping.\n\nOpen questions for user: (a) auto-loop to 'coder' or a dedicated 'fixer' role? (b) opt-in per task_type (proposed) vs global? (c) cap value / mechanism. HOLDING until answered."} | NULL | 2026-07-18T09:55:29Z |