@dfsramos/foreman 0.2.0 → 0.3.0

package/CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to foreman are recorded here. The format follows [Keep a Cha
 
 ## [Unreleased]
 
+## [0.3.0] - 2026-10-26
+
+### Added
+- The Tasks tab can edit a task already on the branch: each row has an **Edit** button that loads its title, description and priority into the form and saves them as one commit (`Edit FM-12: <title>.`). The state, claim and log are kept, and the file keeps its name, so the identifier and the agent's branch do not change.
+
 ## [0.2.0] - 2026-10-26
 
 The first public release, installed with `npm install -g @dfsramos/foreman`.

package/README.md

@@ -211,7 +211,7 @@ Foreman: picked up. Claude Code is working on it.
 
 **Using `main` instead.** Branch can be set to `main`, or any other existing branch, and the task files then live in that branch's tree; but every claim, comment, lease renewal and state move is then a commit on `main`, and every workflow with `on: push` and no branch filter runs on each of them. The orphan branch carries no workflow files, so GitHub Actions does not run on it; a CI system that triggers on every branch of the repository does, and needs a branch filter.
 
-**Tasks tab.** The console's **Tasks** tab lists a Git branch project's tasks grouped by state, shows which machine holds the project's lease, and has a form that creates a task file on the branch (one commit, `Add FM-13: <title>.`). It is the way to add a task from a machine that has no worktree on the task branch.
+**Tasks tab.** The console's **Tasks** tab lists a Git branch project's tasks grouped by state, shows which machine holds the project's lease, and has a form that creates a task file on the branch (one commit, `Add FM-13: <title>.`) and an **Edit** button on each row that changes a task's title, description and priority (one commit, `Edit FM-13: <title>.`; the file keeps its name, state, claim and log). It is the way to add or change a task from a machine that has no worktree on the task branch.
 
 ## Importing from Linear
 

package/dist/server/tasks.js

@@ -1,4 +1,4 @@
-import { createTask } from '../sources/git-import.js';
+import { createTask, updateTask } from '../sources/git-import.js';
 import { createGitSource } from '../sources/git.js';
 const MAX_TITLE_LENGTH = 200;
 const PRIORITY_RANGE = { min: 0, max: 4 };
@@ -7,7 +7,7 @@ const messageOf = (error) => (error instanceof Error ? error.message : String(er
 /** A project's git source, opened for one request; the store behind it is one per repository and branch, shared with the runner's. */
 const gitProjectTasks = (dataDir) => (project) => {
     const source = createGitSource(project.sourceSettings, { repoPath: project.repoPath, dataDir });
-    return { inspect: () => source.inspect(), create: (input) => createTask(source, input) };
+    return { inspect: () => source.inspect(), create: (input) => createTask(source, input), update: (id, input) => updateTask(source, id, input) };
 };
 /** A whole number in `range`, from a number or a string of digits as the console's select sends it; null otherwise. */
 const wholeNumber = (value, range) => {
@@ -38,7 +38,7 @@ function parseNewTask(input) {
         return { errors };
     return { task: { title, description: description === '' ? null : description, priority: priority } };
 }
-/** Routes that list a git-source project's task files with its lease, and add a task file to its branch. */
+/** Routes that list a git-source project's task files with its lease, and add or edit a task file on its branch. */
 export function registerTaskRoutes(app, { db, dataDir, projectTasks = gitProjectTasks(dataDir) }) {
     /** The project's tasks, or why the request cannot have them. */
     const tasksOf = (id) => {
@@ -74,4 +74,21 @@ export function registerTaskRoutes(app, { db, dataDir, projectTasks = gitProject
             return reply.code(502).send({ error: messageOf(error) });
         }
     });
+    app.patch('/api/projects/:id/tasks/:taskId', async (request, reply) => {
+        const target = tasksOf(request.params.id);
+        if ('error' in target)
+            return reply.code(target.status).send({ error: target.error });
+        if (!/^\d+$/.test(request.params.taskId))
+            return reply.code(404).send({ error: 'Task not found' });
+        const parsed = parseNewTask(request.body);
+        if ('errors' in parsed)
+            return reply.code(400).send({ errors: parsed.errors });
+        try {
+            return await target.tasks.update(request.params.taskId, parsed.task);
+        }
+        catch (error) {
+            const message = messageOf(error);
+            return reply.code(/was not found in/.test(message) ? 404 : 502).send({ error: message });
+        }
+    });
 }

package/dist/sources/git-import.js

@@ -1,6 +1,6 @@
 import { importFooter } from '../imports/destination.js';
 import { createGitSource } from './git.js';
-import { identifierOf, newTaskFile, parseTaskName, serialiseTaskFile } from './taskfile.js';
+import { identifierOf, newTaskFile, parseTaskFile, parseTaskName, serialiseTaskFile, withContent } from './taskfile.js';
 /** How often a create is retried when another machine takes the number it was about to use. */
 const MAX_ATTEMPTS = 5;
 /** A time as the files carry it: ISO 8601 in UTC, to the second. */
@@ -29,6 +29,31 @@ export async function createTask(source, input, now = () => new Date()) {
     }
     throw new Error(`Gave up after ${MAX_ATTEMPTS} attempts: the next task number in ${source.directory} kept being taken.`);
 }
+/**
+ * Rewrites the title, description and priority of task `id` and pushes it under `Edit <identifier>: <title>.`.
+ * The file is read again on the synced tree inside the transaction, so a state change or a claim
+ * another machine pushed in between is kept and only the three fields are replaced. Throws when the
+ * task is not on the branch.
+ */
+export async function updateTask(source, id, input) {
+    if (input.title.trim() === '')
+        throw new Error('A task needs a title.');
+    const store = await source.store();
+    const number = Number(id);
+    const identifier = identifierOf(source.prefix, number);
+    const name = (await source.readAll()).find((file) => file.number === number)?.name;
+    if (!name)
+        throw new Error(`Task ${identifier} was not found in ${source.directory}`);
+    const path = `${source.directory}/${name}`;
+    const title = input.title.replace(/\s+/g, ' ').trim();
+    await store.transact(`Edit ${identifier}: ${title}.`, (tree) => {
+        const text = tree.read(path);
+        if (text === null)
+            throw new Error(`Task ${identifier} was not found in ${source.directory}`);
+        return { [path]: serialiseTaskFile(withContent(parseTaskFile(name, text), input)) };
+    });
+    return { identifier, name };
+}
 /**
  * A git source as the place imported tasks are copied into. A task file has no field for an
  * external id, so the description ends with a line naming the origin, and that line is what later

package/dist/sources/git.js

@@ -276,6 +276,7 @@ export function createGitSource(settings, context, deps = {}) {
                     id: String(file.number),
                     identifier: identifier(file.number),
                     title: file.title,
+                    description: file.description,
                     state: file.state,
                     priority: file.priority,
                     claimedBy: metaOf(file, 'claimed_by'),

package/dist/sources/taskfile.js

@@ -150,6 +150,26 @@ export function appendLog(file, at, text) {
     const body = `${opened}\n\n### ${at}\n\n${text.trim()}\n`;
     return { ...file, body, ...derive(body, file.slug) };
 }
+/**
+ * The file with a new title, description and priority. Only the title line and the description
+ * under it are rewritten: whatever precedes the title line, the `## Log` section and every other
+ * frontmatter key stay as they were. The file name is kept, so the identifier and the link to the
+ * file do not move when the title does.
+ */
+export function withContent(file, input) {
+    const title = input.title.replace(/\s+/g, ' ').trim();
+    if (title === '')
+        throw new TaskFileError(file.name, 'a task needs a title');
+    const description = input.description?.trim() ?? '';
+    const lines = linesOf(file.body);
+    const { titleAt, logAt } = layoutOf(lines);
+    const before = titleAt === -1 ? [] : lines.slice(0, titleAt);
+    const after = logAt === lines.length ? [] : ['', ...lines.slice(logAt)];
+    const written = description === '' ? [`# ${title}`] : [`# ${title}`, '', ...description.split(/\r?\n/)];
+    // A body that began with the frontmatter's closing line keeps its leading blank line.
+    const body = [...(titleAt === -1 ? [''] : []), ...before, ...written, ...after, ...(after.length === 0 ? [''] : [])].join('\n');
+    return { ...withMeta(file, { priority: String(input.priority) }), body, ...derive(body, file.slug) };
+}
 /** A fresh `todo` file for a task, as the import and the console create them. */
 export function newTaskFile(number, input) {
     const title = input.title.replace(/\s+/g, ' ').trim();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dfsramos/foreman",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Polling-and-dispatch daemon that hands tracker issues to coding-agent CLIs and follows their pull requests.",
   "license": "MIT",
   "author": "Daniel Ramos",

package/public/style.css

@@ -1586,7 +1586,7 @@ select.input {
 .task-head,
 .task-row {
   display: grid;
-  grid-template-columns: 84px minmax(0, 1.6fr) 96px minmax(0, 1fr) 120px;
+  grid-template-columns: 84px minmax(0, 1.6fr) 96px minmax(0, 1fr) 120px 56px;
   gap: 12px;
   align-items: center;
   padding: 0 16px;

package/public/tasks.js

@@ -1,5 +1,5 @@
 // The Tasks tab: a git-source project's task files grouped by state, the machine that holds the
-// project's lease, and a form that adds a task as a new file on the branch. Everything is read
+// project's lease, and a form that adds a task as a new file on the branch or edits one already there. Everything is read
 // from the branch as it is on the remote, from any machine and lease or not: only polling is
 // gated, so this is where a person sees which machine holds a project and which holds each task.
 // All text from the files is inserted as text nodes.
@@ -16,7 +16,8 @@ const STATES = [
   ['done', 'Done'],
 ];
 
-const emptyForm = () => ({ title: '', description: '', priority: '0', errors: {}, busy: false });
+/** `editing` is the id of the task the form changes, or null when it adds a new one. */
+const emptyForm = () => ({ editing: null, title: '', description: '', priority: '0', errors: {}, busy: false });
 
 const view = {
   projectId: null,
@@ -61,6 +62,8 @@ export function tasksTab(ctx) {
     view.data = null;
     view.error = null;
     view.notice = null;
+    // An edit belongs to the task of the project it was started on; a draft of a new task carries over.
+    if (view.form.editing) view.form = emptyForm();
     view.form.errors = {};
   }
 
@@ -150,7 +153,7 @@ export function tasksTab(ctx) {
       h('summary', null, h('span', { class: 'task-state' }, name), h('span', { class: 'task-count' }, String(tasks.length))),
       tasks.length === 0
         ? h('div', { class: 'task-empty' }, 'None.')
-        : [h('div', { class: 'task-head' }, ...['Task', 'Title', 'Priority', 'Claimed by', 'Pull request'].map((label) => h('span', null, label))), tasks.map(row)],
+        : [h('div', { class: 'task-head' }, ...['Task', 'Title', 'Priority', 'Claimed by', 'Pull request', ''].map((label) => h('span', null, label))), tasks.map(row)],
     );
     // Folding is remembered across refreshes, not across visits to the tab.
     details.addEventListener('toggle', () => {
@@ -169,9 +172,19 @@ export function tasksTab(ctx) {
       h('span', null, PRIORITY_NAMES[task.priority] ?? '—'),
       task.claimedBy ? h('span', { class: 'ellipsis' }, task.claimedBy, h('span', { class: 'faint' }, ` since ${fmtDateTime(task.claimedAt)}`)) : h('span', { class: 'faint' }, '—'),
       task.pr ? externalLink(task.pr, prLabel(task.pr)) : h('span', { class: 'faint' }, '—'),
+      button('Edit', { onclick: () => edit(task) }),
     );
   }
 
+  /** Loads a task into the form; the form takes the place of the "new task" one until it is saved or cancelled. */
+  function edit(task) {
+    view.form = { ...emptyForm(), editing: task.id, title: task.title, description: task.description ?? '', priority: String(task.priority) };
+    view.notice = null;
+    ctx.rerender();
+    document.getElementById('task-title')?.scrollIntoView({ block: 'center' });
+    document.getElementById('task-title')?.focus();
+  }
+
   /* ---------- new task ---------- */
 
   function formCard() {
@@ -208,45 +221,58 @@ export function tasksTab(ctx) {
       form.errors = {};
       view.notice = null;
       ctx.rerender();
-      const response = await api('POST', `/api/projects/${view.projectId}/tasks`, {
-        title: form.title,
-        description: form.description,
-        priority: Number(form.priority),
-      });
+      const editing = form.editing;
+      const body = { title: form.title, description: form.description, priority: Number(form.priority) };
+      const response = editing
+        ? await api('PATCH', `/api/projects/${view.projectId}/tasks/${editing}`, body)
+        : await api('POST', `/api/projects/${view.projectId}/tasks`, body);
       if (view.form !== form) return;
       form.busy = false;
       if (response.ok) {
-        view.notice = { tone: 'ok', title: `Added ${response.data.identifier}`, text: `${response.data.name} is on the branch, waiting as todo.` };
+        view.notice = editing
+          ? { tone: 'ok', title: `Saved ${response.data.identifier}`, text: `${response.data.name} is updated on the branch.` }
+          : { tone: 'ok', title: `Added ${response.data.identifier}`, text: `${response.data.name} is on the branch, waiting as todo.` };
         view.form = emptyForm();
         load();
       } else if (response.data?.errors) {
         form.errors = response.data.errors;
-        if (form.errors._) view.notice = { tone: 'fail', title: 'Not added', text: form.errors._ };
+        if (form.errors._) view.notice = { tone: 'fail', title: editing ? 'Not saved' : 'Not added', text: form.errors._ };
       } else {
-        view.notice = { tone: 'fail', title: 'Not added', text: response.data?.error ?? `Request failed (${response.status})` };
+        view.notice = { tone: 'fail', title: editing ? 'Not saved' : 'Not added', text: response.data?.error ?? `Request failed (${response.status})` };
       }
       ctx.rerender();
     }
 
+    function cancel() {
+      view.form = emptyForm();
+      ctx.rerender();
+    }
+
     showNotice();
     return h(
       'section',
       { class: 'card' },
-      h('div', { class: 'card-head' }, h('h2', null, 'New task'), h('span', { class: 'sub' }, 'One todo file, pushed as one commit')),
+      h(
+        'div',
+        { class: 'card-head' },
+        h('h2', null, form.editing ? 'Edit task' : 'New task'),
+        h('span', { class: 'sub' }, form.editing ? 'Rewrites the title, description and priority; state and log stay' : 'One todo file, pushed as one commit'),
+      ),
       noticeLine,
       h(
         'div',
         { class: 'fields' },
-        field('Title', 'title', 'task-title', title, 'Names the file: <number>-<slug of the title>.md.'),
+        field('Title', 'title', 'task-title', title, form.editing ? 'The file name keeps the old title.' : 'Names the file: <number>-<slug of the title>.md.'),
         field('Priority', 'priority', 'task-priority', priority, 'Urgent tasks are picked up first.'),
         field('Description', 'description', 'task-description', description, 'Markdown. What the agent is told to do.', true),
       ),
       h(
         'div',
         { class: 'save-bar' },
-        h('span', { class: 'note' }, 'Needs no lease: any machine can add a task.'),
+        h('span', { class: 'note' }, form.editing ? 'The file keeps its name, so the identifier and branch do not change.' : 'Needs no lease: any machine can add a task.'),
         h('span', { class: 'spacer' }),
-        button(form.busy ? 'Adding…' : 'Add task', { variant: 'primary', disabled: form.busy || !view.data, onclick: () => void submit() }),
+        form.editing ? button('Cancel', { disabled: form.busy, onclick: cancel }) : null,
+        button(form.busy ? (form.editing ? 'Saving…' : 'Adding…') : form.editing ? 'Save changes' : 'Add task', { variant: 'primary', disabled: form.busy || !view.data, onclick: () => void submit() }),
       ),
     );
   }