placeholder -- real content is PH3 (v_dashboard, U-025)
" }, "#/tasks": { render: () => "placeholder -- real content is PH3 (v_task_board, U-025)
" }, "#/settings": { render: () => "placeholder -- real content is PH3 (U-030)
" }, "#/setup": { render: renderProjectSetup, mount: mountProjectSetup }, + "#/goals/new": { render: renderGoalComposer, mount: mountGoalComposer }, + "#/bugs/new": { render: renderBugFeedback, mount: mountBugFeedback }, }; function render() { @@ -106,6 +123,7 @@ function mountProjectSetup() { }); const body = await resp.json(); if (resp.status === 200) { + setCurrentProjectId(body.project_id); window.location.hash = "#/"; return; } @@ -122,3 +140,176 @@ function mountProjectSetup() { confirmBanner.style.display = "none"; }); } + +// ---- shared: pages that need a "current project" to already exist ---- + +function renderNoProjectSelected() { + return ` +" + body.task_id + "（" + body.status + "）。"
+ + '返回仪表盘';
+ successBox.style.display = "block";
+ });
+}
+
+// ---- v_bug_feedback (02-V §2.4): report a bug ----
+
+function renderBugFeedback() {
+ if (!getCurrentProjectId()) return renderNoProjectSelected();
+ return `
+
+ " + body.bug_id + "）。"
+ + '返回仪表盘';
+ successBox.style.display = "block";
+ });
+}
diff --git a/frontend/style.css b/frontend/style.css
index c9b8538..212d665 100644
--- a/frontend/style.css
+++ b/frontend/style.css
@@ -6,7 +6,8 @@ body { font-family: system-ui, sans-serif; margin: 0; }
#nav a.active { font-weight: bold; }
#app { padding: 1rem; }
-/* U-022: v_project_setup -- class names match docs/wireframes/v_project_setup.html */
+/* U-022/U-023: v_project_setup/v_goal_composer/v_bug_feedback -- class
+ names match docs/wireframes/v_*.html */
.top-back a { color: #6b7280; text-decoration: none; }
.page { padding: 28px; max-width: 560px; margin: 0 auto; }
.page h1 { font-size: 22px; margin: 0 0 4px; }
@@ -14,7 +15,7 @@ body { font-family: system-ui, sans-serif; margin: 0; }
.panel { border: 1px solid #e5e7eb; border-radius: 10px; padding: 24px; background: #f9fafb; }
.field { margin-bottom: 18px; }
.field label { display: block; font-size: 13px; font-weight: 600; margin-bottom: 6px; }
-.field input { width: 100%; padding: 9px 10px; border: 1px solid #e5e7eb; border-radius: 6px; font-size: 14px; }
+.field input, .field textarea { width: 100%; padding: 9px 10px; border: 1px solid #e5e7eb; border-radius: 6px; font-size: 14px; font-family: inherit; }
.field input[readonly] { background: #f3f4f6; color: #6b7280; }
.field .hint { font-size: 12px; color: #6b7280; margin-top: 5px; }
.btn { display: inline-block; padding: 9px 16px; border-radius: 6px; border: 1px solid #e5e7eb; background: #fff; font-size: 14px; cursor: pointer; }
@@ -22,3 +23,5 @@ body { font-family: system-ui, sans-serif; margin: 0; }
.btn-row { display: flex; gap: 10px; margin-top: 22px; }
.confirm-banner { background: #fffbeb; border: 1px solid #f59e0b; border-radius: 8px; padding: 12px 14px; font-size: 13px; margin-top: 16px; }
.error-banner { background: #fef2f2; border: 1px solid #ef4444; border-radius: 8px; padding: 12px 14px; font-size: 13px; margin-bottom: 16px; }
+.success-banner { background: #f0fdf4; border: 1px solid #22c55e; border-radius: 8px; padding: 12px 14px; font-size: 13px; margin-bottom: 16px; }
+.success-banner a { margin-left: 8px; color: #4f46e5; }
diff --git a/src/backend_server.py b/src/backend_server.py
index 1311e03..497ff38 100644
--- a/src/backend_server.py
+++ b/src/backend_server.py
@@ -8,10 +8,12 @@ platforms -- U-007 shipped macOS only via `osascript`; U-022 adds Windows
via a `powershell` FolderBrowserDialog script and Linux via `zenity`/
`kdialog`, whichever is installed), `c_project_create` (03-C §3.0, wired to
the U-002 `project_scaffold.create_project()` function that had no HTTP
-route until now) for `v_project_setup`'s submit flow, and serving the U-008
-frontend shell as static files (no build step, 05 §5.7: whitelisted by exact
-path, not a generic directory walker, so there's no path-traversal surface
-to reason about).
+route until now) for `v_project_setup`'s submit flow, `c_task_create`/
+`c_bug_report_create` (03-C §3.1/§3.1.1, U-023) for `v_goal_composer`/
+`v_bug_feedback`'s submit flows, and serving the U-008 frontend shell as
+static files (no build step, 05 §5.7: whitelisted by exact path, not a
+generic directory walker, so there's no path-traversal surface to reason
+about).
"""
import json
@@ -21,7 +23,7 @@ import subprocess
import threading
from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer
-from src import paths, project_scaffold, schema
+from src import bug_record, paths, project_scaffold, schema, task_create
_FRONTEND_DIR = os.path.join(os.path.dirname(os.path.dirname(os.path.abspath(__file__))), "frontend")
@@ -121,6 +123,10 @@ class _Handler(BaseHTTPRequestHandler):
self._handle_pick_folder()
elif self.path == "/api/projects":
self._handle_project_create()
+ elif self.path == "/api/tasks":
+ self._handle_task_create()
+ elif self.path == "/api/bug-reports":
+ self._handle_bug_report_create()
else:
self._send_json(404, {"error": "NOT_FOUND"})
@@ -130,12 +136,19 @@ class _Handler(BaseHTTPRequestHandler):
except DialogUnavailableError as e:
self._send_json(422, {"error": "DIALOG_UNAVAILABLE", "detail": str(e)})
- def _handle_project_create(self):
+ def _read_json_body(self):
+ """Returns (payload, None) on success, (None, error_response) if the
+ body isn't valid JSON -- caller sends error_response and returns."""
length = int(self.headers.get("Content-Length", 0))
try:
- payload = json.loads(self.rfile.read(length) or b"{}")
+ return json.loads(self.rfile.read(length) or b"{}"), None
except json.JSONDecodeError:
- self._send_json(400, {"error": "INVALID_INPUT", "detail": "body is not valid JSON"})
+ return None, {"error": "INVALID_INPUT", "detail": "body is not valid JSON"}
+
+ def _handle_project_create(self):
+ payload, error = self._read_json_body()
+ if error:
+ self._send_json(400, error)
return
target_folder_path = payload.get("target_folder_path")
if not target_folder_path:
@@ -158,6 +171,44 @@ class _Handler(BaseHTTPRequestHandler):
else:
self._send_json(200, result)
+ def _handle_task_create(self):
+ payload, error = self._read_json_body()
+ if error:
+ self._send_json(400, error)
+ return
+ try:
+ with self.server.conn_lock:
+ result = task_create.create_task(
+ self.server.conn,
+ payload.get("project_id"),
+ payload.get("goal_text"),
+ task_type=payload.get("task_type", "goal"),
+ created_via=payload.get("created_via", "api"),
+ )
+ except task_create.InvalidParamError as e:
+ self._send_json(400, {"error": "INVALID_PARAM", "detail": str(e)})
+ else:
+ self._send_json(200, result)
+
+ def _handle_bug_report_create(self):
+ payload, error = self._read_json_body()
+ if error:
+ self._send_json(400, error)
+ return
+ try:
+ with self.server.conn_lock:
+ result = bug_record.create_bug_report(
+ self.server.conn,
+ payload.get("project_id"),
+ payload.get("description"),
+ repro_steps=payload.get("repro_steps"),
+ related_task_id=payload.get("related_task_id"),
+ )
+ except bug_record.InvalidParamError as e:
+ self._send_json(400, {"error": "INVALID_PARAM", "detail": str(e)})
+ else:
+ self._send_json(200, result)
+
def _send_json(self, status, obj):
body = json.dumps(obj).encode("utf-8")
self.send_response(status)
diff --git a/src/bug_record.py b/src/bug_record.py
index 3bcabd6..4ce96b9 100644
--- a/src/bug_record.py
+++ b/src/bug_record.py
@@ -34,6 +34,48 @@ class InvalidRootCauseCategoryError(Exception):
"""INVALID_ROOT_CAUSE_CATEGORY -- 03-C §3.10."""
+class InvalidParamError(Exception):
+ """INVALID_PARAM -- 03-C §3.1.1: description is empty, or project_id/
+ related_task_id don't reference existing rows."""
+
+
+def create_bug_report(conn, project_id, description, repro_steps=None, related_task_id=None):
+ """c_bug_report_create (03-C §3.1.1): console-submitted human bug report.
+
+ 03-C's own input_contract for this controller lists only
+ {description, repro_steps, related_task_id} -- no project_id -- but
+ execution_event.project_id is NOT NULL (01-M 1.9) and bug_record has no
+ project_id column of its own to derive one from when related_task_id is
+ left empty ("留空则视为新发现的独立问题", 02-V §2.4). Requiring
+ project_id here (like c_task_create already does per its own contract)
+ is the only way to satisfy that constraint for a task-less report; this
+ is a real 03-C gap, not a design choice, flagged rather than silently
+ routed around -- see development-plan.md's 2026-07-31 note on this Unit.
+ """
+ if not description:
+ raise InvalidParamError("description is required")
+ if conn.execute("SELECT 1 FROM project WHERE project_id = ?", (project_id,)).fetchone() is None:
+ raise InvalidParamError("project_id does not exist: " + str(project_id))
+ if related_task_id is not None and conn.execute(
+ "SELECT 1 FROM task WHERE task_id = ?", (related_task_id,)
+ ).fetchone() is None:
+ raise InvalidParamError("related_task_id does not exist: " + str(related_task_id))
+
+ bug_id = str(uuid.uuid4())
+ conn.execute(
+ "INSERT INTO bug_record (bug_id, task_id, source, description, repro_steps, status) "
+ "VALUES (?, ?, 'human', ?, ?, 'open')",
+ (bug_id, related_task_id, description, repro_steps),
+ )
+ conn.execute(
+ "INSERT INTO execution_event (event_id, project_id, task_id, event_type, payload) "
+ "VALUES (?, ?, ?, 'bug_recorded', ?)",
+ (str(uuid.uuid4()), project_id, related_task_id, json.dumps({"bug_id": bug_id, "source": "human"})),
+ )
+ conn.commit()
+ return {"bug_id": bug_id, "status": "open"}
+
+
def create_bug_record(
conn, task_id, description, root_cause_category, root_cause_detail, suggested_route,
repro_steps=None, related_artifact_ids=None, skill_iteration_suggestion_id=None,
diff --git a/src/test_backend_server.py b/src/test_backend_server.py
index c74cec7..a30fa97 100644
--- a/src/test_backend_server.py
+++ b/src/test_backend_server.py
@@ -313,6 +313,82 @@ class ProjectCreateRouteTest(unittest.TestCase):
self.assertTrue(os.path.isdir(os.path.join(non_git, ".git")))
+class TaskAndBugReportRouteTest(unittest.TestCase):
+ """U-023: c_task_create (03-C §3.1) + c_bug_report_create (03-C §3.1.1)
+ wired to real HTTP routes for v_goal_composer/v_bug_feedback."""
+
+ def setUp(self):
+ self.server = backend_server.create_server(db_path=":memory:")
+ self.thread = threading.Thread(target=self.server.serve_forever, daemon=True)
+ self.thread.start()
+ self.server.conn.execute(
+ "INSERT INTO project (project_id, name, target_folder_path, has_git_repo, status) "
+ "VALUES ('proj-1', 'P', '/tmp/proj-1', 1, 'active')"
+ )
+ self.server.conn.commit()
+
+ def tearDown(self):
+ self.server.shutdown()
+ self.server.server_close()
+ self.thread.join(timeout=5)
+
+ def _post(self, path, payload):
+ host, port = self.server.server_address
+ conn = http.client.HTTPConnection(host, port, timeout=5)
+ conn.request("POST", path, body=json.dumps(payload).encode("utf-8"))
+ resp = conn.getresponse()
+ body = json.loads(resp.read())
+ conn.close()
+ return resp.status, body
+
+ def test_creates_task(self):
+ status, body = self._post(
+ "/api/tasks", {"project_id": "proj-1", "goal_text": "make a snake game", "created_via": "human_console"}
+ )
+ self.assertEqual(status, 200)
+ self.assertEqual(body["status"], "queued")
+ self.assertIn("task_id", body)
+
+ def test_empty_goal_text_returns_400(self):
+ status, body = self._post("/api/tasks", {"project_id": "proj-1", "goal_text": ""})
+ self.assertEqual(status, 400)
+ self.assertEqual(body["error"], "INVALID_PARAM")
+
+ def test_nonexistent_project_id_returns_400(self):
+ status, body = self._post("/api/tasks", {"project_id": "no-such-project", "goal_text": "x"})
+ self.assertEqual(status, 400)
+ self.assertEqual(body["error"], "INVALID_PARAM")
+
+ def test_creates_bug_report(self):
+ status, body = self._post(
+ "/api/bug-reports", {"project_id": "proj-1", "description": "snake clips through walls"}
+ )
+ self.assertEqual(status, 200)
+ self.assertEqual(body["status"], "open")
+ self.assertIn("bug_id", body)
+
+ def test_bug_report_with_related_task_id(self):
+ _, task_body = self._post("/api/tasks", {"project_id": "proj-1", "goal_text": "x"})
+ status, body = self._post(
+ "/api/bug-reports",
+ {"project_id": "proj-1", "description": "x", "related_task_id": task_body["task_id"]},
+ )
+ self.assertEqual(status, 200)
+
+ def test_empty_description_returns_400(self):
+ status, body = self._post("/api/bug-reports", {"project_id": "proj-1", "description": ""})
+ self.assertEqual(status, 400)
+ self.assertEqual(body["error"], "INVALID_PARAM")
+
+ def test_bug_report_with_bad_project_id_returns_400_not_500(self):
+ # a raw sqlite3.IntegrityError propagating uncaught here was a real
+ # bug this smoke test caught (stale/bad project_id crashed the
+ # request instead of a clean 400)
+ status, body = self._post("/api/bug-reports", {"project_id": "no-such-project", "description": "x"})
+ self.assertEqual(status, 400)
+ self.assertEqual(body["error"], "INVALID_PARAM")
+
+
class ZeroThirdPartyDependencyTest(unittest.TestCase):
"""05 §5.7: no Flask/FastAPI/Express/Node.js -- checked statically so this
doesn't silently rot if someone adds an import later."""
diff --git a/src/test_bug_record.py b/src/test_bug_record.py
index c58602f..92a97a6 100644
--- a/src/test_bug_record.py
+++ b/src/test_bug_record.py
@@ -70,6 +70,75 @@ class CreateBugRecordTest(BugRecordTestBase):
)
+class CreateBugReportTest(BugRecordTestBase):
+ """U-023: c_bug_report_create (03-C §3.1.1) -- console-submitted human
+ bug reports, distinct from create_bug_record's Validator-judged path."""
+
+ def test_happy_path_creates_open_human_record(self):
+ result = bug_record.create_bug_report(
+ self.conn, project_id="proj-1", description="snake doesn't wrap around edges",
+ )
+ row = self.conn.execute(
+ "SELECT source, status, task_id, description FROM bug_record WHERE bug_id = ?",
+ (result["bug_id"],),
+ ).fetchone()
+ self.assertEqual(row, ("human", "open", None, "snake doesn't wrap around edges"))
+ self.assertEqual(result["status"], "open")
+
+ def test_related_task_id_is_optional_and_stored_when_given(self):
+ result = bug_record.create_bug_report(
+ self.conn, project_id="proj-1", description="x", related_task_id="task-1",
+ )
+ row = self.conn.execute(
+ "SELECT task_id FROM bug_record WHERE bug_id = ?", (result["bug_id"],)
+ ).fetchone()
+ self.assertEqual(row[0], "task-1")
+
+ def test_repro_steps_optional_and_stored_when_given(self):
+ result = bug_record.create_bug_report(
+ self.conn, project_id="proj-1", description="x", repro_steps="1. start game 2. hit wall",
+ )
+ row = self.conn.execute(
+ "SELECT repro_steps FROM bug_record WHERE bug_id = ?", (result["bug_id"],)
+ ).fetchone()
+ self.assertEqual(row[0], "1. start game 2. hit wall")
+
+ def test_empty_description_rejected(self):
+ with self.assertRaises(bug_record.InvalidParamError):
+ bug_record.create_bug_report(self.conn, project_id="proj-1", description="")
+
+ def test_nonexistent_project_id_rejected(self):
+ # a raw FK violation surfacing as an uncaught IntegrityError instead
+ # of this is exactly what a real backend_server.py smoke test caught
+ # (a bad/stale project_id would otherwise 500 the request)
+ with self.assertRaises(bug_record.InvalidParamError):
+ bug_record.create_bug_report(self.conn, project_id="no-such-project", description="x")
+
+ def test_nonexistent_related_task_id_rejected(self):
+ with self.assertRaises(bug_record.InvalidParamError):
+ bug_record.create_bug_report(
+ self.conn, project_id="proj-1", description="x", related_task_id="no-such-task"
+ )
+
+ def test_writes_bug_recorded_execution_event(self):
+ result = bug_record.create_bug_report(self.conn, project_id="proj-1", description="x")
+ row = self.conn.execute(
+ "SELECT project_id, event_type FROM execution_event WHERE payload LIKE ?",
+ ('%' + result["bug_id"] + '%',),
+ ).fetchone()
+ self.assertEqual(row, ("proj-1", "bug_recorded"))
+
+ def test_no_related_task_still_writes_event_with_null_task_id(self):
+ # this is exactly the case 03-C's own input_contract can't satisfy
+ # without project_id -- see create_bug_report's docstring
+ result = bug_record.create_bug_report(self.conn, project_id="proj-1", description="x")
+ row = self.conn.execute(
+ "SELECT task_id FROM execution_event WHERE payload LIKE ?",
+ ('%' + result["bug_id"] + '%',),
+ ).fetchone()
+ self.assertIsNone(row[0])
+
+
class HumanReportedFlowTest(BugRecordTestBase):
def _insert_human_bug(self, bug_id="bug-1", status="open"):
self.conn.execute(