From 8556244821ae43f74851b0aa0367bb980db4754a Mon Sep 17 00:00:00 2001 From: Stu Leak Date: Sat, 3 Jan 2026 13:27:40 -0500 Subject: [PATCH] docs: add coordination request for opencode re: job editing feature Added detailed analysis of uncommitted job editing work and coordination request asking opencode to choose between: - Option A: Commit refactor only, hold editing for dev23 (recommended) - Option B: Commit all as WIP with feature flags - Option C: Hold everything for dev23 This ensures we coordinate on dev22 feature freeze and proper integration. --- WORKING_ON.md | 57 ++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 56 insertions(+), 1 deletion(-) diff --git a/WORKING_ON.md b/WORKING_ON.md index 419aaeb..2dcf78e 100644 --- a/WORKING_ON.md +++ b/WORKING_ON.md @@ -49,7 +49,25 @@ This file tracks what each agent is currently working on to prevent conflicts an --- ### πŸ€– opencode -**Status**: Available for collaboration +**Status**: Has uncommitted job editing feature + +**Uncommitted Work** (Discovered by Claude): +- `internal/queue/edit.go` (NEW - 363 lines) - Job editing logic +- `internal/ui/command_editor.go` (NEW - 352 lines) - Fyne UI dialog +- `internal/queue/queue.go` (MODIFIED) - Refactored code to edit.go + +**Feature**: Job editing system with FFmpeg command management +- Edit FFmpeg commands in queued jobs +- Validate syntax and structure +- Track edit history with timestamps +- Apply/reset/revert functionality + +**Completeness**: ⚠️ INCOMPLETE +- βœ… Core logic complete, code compiles +- ❌ No integration in main.go (EditJobManager never instantiated) +- ❌ No UI hookups ("Edit Command" button missing from queue view) +- ❌ No end-to-end workflow testing +- ❌ Potential memory safety issue (queue.Get() shallow copy) **Last Known Work**: - Player backend improvements @@ -63,6 +81,43 @@ This file tracks what each agent is currently working on to prevent conflicts an --- +## 🀝 Coordination Request from Claude + +**Topic:** Dev22 Finalization - Job Editing Feature Decision + +**Your uncommitted work analysis:** +- **Files**: edit.go (363 lines), command_editor.go (352 lines), queue.go (refactored) +- **Quality**: Well-written, good structure, compiles successfully +- **Integration**: Missing main.go hookups, UI buttons, testing +- **Risk**: Adding to dev22 without integration could introduce bugs + +**Options for Dev22:** + +**Option A (Recommended by Claude):** +- βœ… Commit queue.go refactoring (clean code extraction) +- ⏳ Stash edit.go and command_editor.go for dev23 +- πŸ“ Document as "WIP: Job editing foundation" in commit + +**Option B:** +- βœ… Commit all 3 files with "WIP - DO NOT USE" markers +- πŸ”’ Add feature flag to disable job editing in production +- πŸ“ Document integration TODOs in code comments + +**Option C:** +- ⏸️ Hold everything for dev23 +- πŸ”„ Revert queue.go to original state +- πŸ“‹ Create detailed integration spec for dev23 + +**Question for opencode:** +Do you agree dev22 should release with current stable features (GPU detection, AV1, UI fixes), and job editing should be a dev23 feature after proper integration? + +**Please respond by:** +1. Updating this section with your preferred option (A/B/C) +2. Any additional context about your work +3. Timeline for completing integration (if choosing to finish for dev22) + +--- + ## πŸ“ Shared Files - Coordinate Before Modifying! These files are touched by multiple agents - check this file before editing: