@1mancompany/onemancompany 0.7.32 → 0.7.39

package/frontend/app.js

@@ -7555,6 +7555,7 @@ class AppController {
       { id: 'issues', label: `Issues (${issues.length})` },
       { id: 'kanban', label: 'Kanban' },
       { id: 'roadmap', label: 'Roadmap' },
+      { id: 'reviews', label: `Reviews (${(data.reviews || []).length})` },
       { id: 'activity', label: 'Activity' },
       { id: 'projects', label: `Projects (${projects.length})` },
     ];
@@ -7591,6 +7592,8 @@ class AppController {
       this._renderProductKanban(slug, container, data);
     } else if (tabId === 'roadmap') {
       this._renderProductRoadmap(slug, container);
+    } else if (tabId === 'reviews') {
+      this._renderProductReviews(data.reviews || [], slug, container);
     } else if (tabId === 'activity') {
       this._renderProductActivity(slug, container);
     } else if (tabId === 'projects') {
@@ -7783,6 +7786,21 @@ class AppController {
       unitEl.textContent = unit;
       this._makeKrFieldEditable(unitEl, slug, kr.id, 'unit');
 
+      // Delete KR button
+      const delKrBtn = document.createElement('button');
+      delKrBtn.className = 'kr-remove-btn';
+      delKrBtn.innerHTML = '×';
+      delKrBtn.title = 'Delete KR';
+      delKrBtn.addEventListener('click', async () => {
+        if (!confirm(`Delete KR "${kr.title}"?`)) return;
+        try {
+          const r = await fetch(`/api/product/${encodeURIComponent(slug)}/kr/${encodeURIComponent(kr.id)}`, { method: 'DELETE' });
+          if (!r.ok) throw new Error(`HTTP ${r.status}`);
+          this._showToast('KR deleted', 'success');
+          this._openProductDetail(slug);
+        } catch (err) { this._showToast(`Failed: ${err.message}`, 'error'); }
+      });
+
       krRow.appendChild(titleEl);
       krRow.appendChild(document.createTextNode(': '));
       krRow.appendChild(currentEl);
@@ -7791,6 +7809,7 @@ class AppController {
       krRow.appendChild(unitEl);
       krRow.appendChild(document.createTextNode(` (${pct.toFixed(0)}%)`));
       krRow.appendChild(progTrack);
+      krRow.appendChild(delKrBtn);
       krList.appendChild(krRow);
     }
 
@@ -7803,11 +7822,12 @@ class AppController {
     container.appendChild(krList);
 
     // Version History
+    const verLabel = document.createElement('div');
+    verLabel.className = 'product-section-label';
+    verLabel.textContent = 'Version History';
+    container.appendChild(verLabel);
+
     if (versions.length > 0) {
-      const verLabel = document.createElement('div');
-      verLabel.className = 'product-section-label';
-      verLabel.textContent = 'Version History';
-      container.appendChild(verLabel);
       const verList = document.createElement('div');
       verList.className = 'product-version-list';
       for (const v of versions) {
@@ -7826,6 +7846,13 @@ class AppController {
       }
       container.appendChild(verList);
     }
+
+    // Release Version button
+    const releaseBtn = document.createElement('button');
+    releaseBtn.className = 'btn-small';
+    releaseBtn.textContent = '+ Release Version';
+    releaseBtn.addEventListener('click', () => this._showReleaseVersionForm(container, slug));
+    container.appendChild(releaseBtn);
   }
 
   _makeEditable(el, fieldName, slug) {
@@ -8152,10 +8179,40 @@ class AppController {
       ${labels ? `<div>Labels: ${labels}</div>` : ''}
       ${issue.created_by ? `<div>Created by: ${this._escHtml(issue.created_by)}</div>` : ''}
       ${issue.resolution ? `<div>Resolution: ${this._escHtml(issue.resolution)}</div>` : ''}
-      ${issue.sprint ? `<div>Sprint: ${this._escHtml(issue.sprint)}</div>` : ''}
     `;
     body.appendChild(metaEl);
 
+    // Sprint picker (dropdown)
+    const sprintRow = document.createElement('div');
+    sprintRow.textContent = 'Sprint: ';
+    const sprintSel = document.createElement('select');
+    sprintSel.className = 'form-input';
+    sprintSel.style.width = 'auto';
+    sprintSel.style.display = 'inline';
+    sprintSel.innerHTML = '<option value="">No Sprint</option>';
+    fetch(`/api/product/${encodeURIComponent(slug)}/sprints`)
+      .then(r => r.json())
+      .then(sprints => {
+        for (const s of sprints.filter(s => s.status !== 'closed')) {
+          const opt = document.createElement('option');
+          opt.value = s.id;
+          opt.textContent = `${s.name}${s.status === 'active' ? ' (active)' : ''}`;
+          sprintSel.appendChild(opt);
+        }
+        sprintSel.value = issue.sprint || '';
+      })
+      .catch(err => console.warn('Failed to load sprints:', err));
+    sprintSel.addEventListener('change', () => {
+      fetch(`/api/product/${encodeURIComponent(slug)}/issue/${encodeURIComponent(issue.id)}`, {
+        method: 'PUT',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ sprint: sprintSel.value || '' }),
+      }).then(r => { if (!r.ok) throw new Error(`HTTP ${r.status}`); })
+        .catch(err => this._showToast(`Failed: ${err.message}`, 'error'));
+    });
+    sprintRow.appendChild(sprintSel);
+    body.appendChild(sprintRow);
+
     const assignRow = document.createElement('div');
     assignRow.textContent = 'Assignee: ';
     const assignSel = document.createElement('select');
@@ -8402,11 +8459,26 @@ class AppController {
           <option value="P2" selected>P2</option><option value="P3">P3</option>
         </select>
         <input type="number" class="form-input issue-new-sp" placeholder="Story pts" style="width:60px" />
-        <input type="text" class="form-input issue-new-sprint" placeholder="Sprint" style="width:80px" />
+        <select class="form-input issue-new-sprint" style="width:auto">
+          <option value="">No Sprint</option>
+        </select>
         <button class="btn-small issue-new-save">Create</button>
         <button class="kr-remove-btn issue-new-cancel">&times;</button>
       </div>
     `;
+    // Populate sprint picker from available sprints
+    const sprintSel = row.querySelector('.issue-new-sprint');
+    fetch(`/api/product/${encodeURIComponent(slug)}/sprints`)
+      .then(r => r.json())
+      .then(sprints => {
+        for (const s of sprints.filter(s => s.status !== 'closed')) {
+          const opt = document.createElement('option');
+          opt.value = s.id;
+          opt.textContent = `${s.name}${s.status === 'active' ? ' (active)' : ''}`;
+          sprintSel.appendChild(opt);
+        }
+      })
+      .catch(err => console.warn('Failed to load sprints:', err));
     row.querySelector('.issue-new-cancel').addEventListener('click', () => row.remove());
     row.querySelector('.issue-new-save').addEventListener('click', async () => {
       const title = row.querySelector('.issue-new-title').value.trim();
@@ -8607,6 +8679,11 @@ class AppController {
             const actions = document.createElement('div');
             actions.className = 'sprint-actions';
             if (s.status === 'planning') {
+              const editBtn = document.createElement('button');
+              editBtn.className = 'sprint-action-btn';
+              editBtn.textContent = 'Edit';
+              editBtn.addEventListener('click', () => this._showEditSprintForm(bar, slug, s));
+              actions.appendChild(editBtn);
               const startBtn = document.createElement('button');
               startBtn.className = 'sprint-action-btn';
               startBtn.textContent = 'Start';
@@ -8758,10 +8835,26 @@ class AppController {
       </div>
       <div class="sprint-form-row">
         <input type="number" class="form-input sprint-new-capacity" placeholder="Capacity (pts)" style="width:80px" />
+        <span class="sprint-suggested-capacity" style="color:var(--text-dim);font-size:calc(5px + var(--font-boost));margin-left:4px"></span>
         <button class="btn-small sprint-save-btn">Create</button>
         <button class="kr-remove-btn sprint-cancel-btn">&times;</button>
       </div>
     `;
+    // Show suggested capacity if available
+    fetch(`/api/product/${encodeURIComponent(slug)}/sprint/suggest-capacity`)
+      .then(r => r.json())
+      .then(d => {
+        if (d.suggested_capacity != null) {
+          const hint = form.querySelector('.sprint-suggested-capacity');
+          hint.textContent = `(suggested: ${d.suggested_capacity} pts)`;
+          hint.style.cursor = 'pointer';
+          hint.title = 'Click to use suggested capacity';
+          hint.addEventListener('click', () => {
+            form.querySelector('.sprint-new-capacity').value = d.suggested_capacity;
+          });
+        }
+      })
+      .catch(err => console.warn('Failed to load suggested capacity:', err));
     // Default dates: today → +14 days
     const today = new Date();
     const end = new Date(today);
@@ -8796,6 +8889,51 @@ class AppController {
     form.querySelector('.sprint-new-name').focus();
   }
 
+  _showEditSprintForm(barEl, slug, sprint) {
+    if (barEl.querySelector('.sprint-inline-add')) return;
+    const form = document.createElement('div');
+    form.className = 'sprint-inline-add';
+    form.innerHTML = `
+      <input type="text" class="form-input sprint-edit-name" value="${this._escHtml(sprint.name)}" />
+      <input type="text" class="form-input sprint-edit-goal" value="${this._escHtml(sprint.goal || '')}" placeholder="Goal" />
+      <div class="sprint-form-row">
+        <label style="color:var(--text-dim);font-size:calc(5px + var(--font-boost))">Start:</label>
+        <input type="date" class="form-input sprint-edit-start" value="${sprint.start_date || ''}" style="width:auto" />
+        <label style="color:var(--text-dim);font-size:calc(5px + var(--font-boost))">End:</label>
+        <input type="date" class="form-input sprint-edit-end" value="${sprint.end_date || ''}" style="width:auto" />
+      </div>
+      <div class="sprint-form-row">
+        <input type="number" class="form-input sprint-edit-capacity" value="${sprint.capacity || ''}" placeholder="Capacity" style="width:80px" />
+        <button class="btn-small sprint-edit-save">Save</button>
+        <button class="kr-remove-btn sprint-edit-cancel">&times;</button>
+      </div>
+    `;
+    form.querySelector('.sprint-edit-cancel').addEventListener('click', () => form.remove());
+    form.querySelector('.sprint-edit-save').addEventListener('click', async () => {
+      const updates = {
+        name: form.querySelector('.sprint-edit-name').value.trim(),
+        goal: form.querySelector('.sprint-edit-goal').value.trim(),
+        start_date: form.querySelector('.sprint-edit-start').value,
+        end_date: form.querySelector('.sprint-edit-end').value,
+      };
+      const cap = parseInt(form.querySelector('.sprint-edit-capacity').value);
+      if (!isNaN(cap)) updates.capacity = cap;
+      if (!updates.name) { this._showToast('Sprint name is required', 'warning'); return; }
+      try {
+        const r = await fetch(`/api/product/${encodeURIComponent(slug)}/sprint/${encodeURIComponent(sprint.id)}`, {
+          method: 'PUT',
+          headers: { 'Content-Type': 'application/json' },
+          body: JSON.stringify(updates),
+        });
+        if (!r.ok) { const err = await r.json(); throw new Error(err.detail || r.statusText); }
+        this._showToast('Sprint updated', 'success');
+        this._openProductDetail(slug);
+      } catch (err) { this._showToast(`Update failed: ${err.message}`, 'error'); }
+    });
+    barEl.appendChild(form);
+    form.querySelector('.sprint-edit-name').focus();
+  }
+
   // ---------------------------------------------------------------------------
   // Activity Feed Tab
   // ---------------------------------------------------------------------------
@@ -8874,6 +9012,207 @@ class AppController {
       .catch(err => { container.innerHTML = `<div class="error-text">Failed to load activity: ${err.message}</div>`; });
   }
 
+  // ---------------------------------------------------------------------------
+  // Reviews Tab
+  // ---------------------------------------------------------------------------
+
+  _renderProductReviews(reviews, slug, container) {
+    container.innerHTML = '';
+
+    // Create Review button
+    const toolbar = document.createElement('div');
+    toolbar.className = 'issue-toolbar';
+    const createBtn = document.createElement('button');
+    createBtn.className = 'btn-small';
+    createBtn.textContent = '+ Create Review';
+    createBtn.addEventListener('click', async () => {
+      try {
+        const r = await fetch(`/api/product/${encodeURIComponent(slug)}/review`, {
+          method: 'POST',
+          headers: { 'Content-Type': 'application/json' },
+          body: JSON.stringify({ trigger: 'manual', owner: '' }),
+        });
+        if (!r.ok) throw new Error(`HTTP ${r.status}`);
+        this._showToast('Review created', 'success');
+        this._openProductDetail(slug);
+      } catch (err) { this._showToast(`Failed: ${err.message}`, 'error'); }
+    });
+    toolbar.appendChild(createBtn);
+    container.appendChild(toolbar);
+
+    if (!reviews.length) {
+      const emptyMsg = document.createElement('div');
+      emptyMsg.className = 'task-empty';
+      emptyMsg.textContent = 'No reviews yet.';
+      container.appendChild(emptyMsg);
+      return;
+    }
+
+    // Group: open first, then completed
+    const open = reviews.filter(r => r.status === 'open');
+    const completed = reviews.filter(r => r.status === 'completed');
+
+    for (const group of [{ label: 'Open', items: open }, { label: 'Completed', items: completed }]) {
+      if (!group.items.length) continue;
+      const heading = document.createElement('div');
+      heading.className = 'product-section-label';
+      heading.textContent = `${group.label} (${group.items.length})`;
+      container.appendChild(heading);
+
+      for (const rev of group.items) {
+        const card = document.createElement('div');
+        card.className = `review-card ${rev.status === 'completed' ? 'review-completed' : ''}`;
+
+        const header = document.createElement('div');
+        header.className = 'review-card-header';
+        const trigger = rev.trigger || 'manual';
+        const dateStr = rev.created_at ? new Date(rev.created_at).toLocaleDateString() : '';
+        header.innerHTML = `<span class="review-trigger">${this._escHtml(trigger)}</span> <span class="review-date">${dateStr}</span>`;
+        if (rev.owner) header.innerHTML += ` <span class="review-owner">Owner: ${this._escHtml(rev.owner)}</span>`;
+        card.appendChild(header);
+
+        // Checklist items
+        const itemsList = document.createElement('div');
+        itemsList.className = 'review-items';
+        for (const item of (rev.items || [])) {
+          const row = document.createElement('div');
+          row.className = 'review-item-row';
+          const checkbox = document.createElement('input');
+          checkbox.type = 'checkbox';
+          checkbox.checked = !!item.checked;
+          checkbox.disabled = rev.status === 'completed';
+          checkbox.addEventListener('change', async () => {
+            try {
+              const r = await fetch(`/api/product/${encodeURIComponent(slug)}/review/${encodeURIComponent(rev.id)}/item/${encodeURIComponent(item.key)}`, {
+                method: 'PUT',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ checked: checkbox.checked }),
+              });
+              if (!r.ok) throw new Error(`HTTP ${r.status}`);
+            } catch (err) {
+              checkbox.checked = !checkbox.checked;
+              this._showToast(`Failed: ${err.message}`, 'error');
+            }
+          });
+          const label = document.createElement('span');
+          label.className = item.checked ? 'review-item-checked' : '';
+          label.textContent = item.label || item.key;
+          row.appendChild(checkbox);
+          row.appendChild(label);
+          itemsList.appendChild(row);
+        }
+        card.appendChild(itemsList);
+
+        // Complete button for open reviews
+        if (rev.status === 'open') {
+          const completeBtn = document.createElement('button');
+          completeBtn.className = 'btn-small';
+          completeBtn.textContent = 'Complete Review';
+          completeBtn.addEventListener('click', async () => {
+            try {
+              const r = await fetch(`/api/product/${encodeURIComponent(slug)}/review/${encodeURIComponent(rev.id)}/complete`, { method: 'POST' });
+              if (!r.ok) { const err = await r.json(); throw new Error(err.detail || r.statusText); }
+              this._showToast('Review completed', 'success');
+              this._openProductDetail(slug);
+            } catch (err) { this._showToast(`Failed: ${err.message}`, 'error'); }
+          });
+          card.appendChild(completeBtn);
+        }
+
+        container.appendChild(card);
+      }
+    }
+  }
+
+  // ---------------------------------------------------------------------------
+  // Release Version Form
+  // ---------------------------------------------------------------------------
+
+  _showReleaseVersionForm(container, slug) {
+    if (container.querySelector('.release-form')) return;
+    const form = document.createElement('div');
+    form.className = 'release-form sprint-inline-add';
+
+    // Show done issues that can be released
+    fetch(`/api/product/${encodeURIComponent(slug)}/detail`)
+      .then(r => r.json())
+      .then(data => {
+        const doneIssues = (data.issues || []).filter(i => i.status === 'done');
+        if (!doneIssues.length) {
+          form.innerHTML = '<div class="task-empty">No issues in DONE status to release.</div>';
+          const closeBtn = document.createElement('button');
+          closeBtn.className = 'kr-remove-btn';
+          closeBtn.innerHTML = '&times;';
+          closeBtn.addEventListener('click', () => form.remove());
+          form.appendChild(closeBtn);
+          return;
+        }
+
+        const label = document.createElement('div');
+        label.style.cssText = 'color:var(--text-dim);font-size:calc(5px + var(--font-boost));margin-bottom:4px';
+        label.textContent = `Select issues to include in release (${doneIssues.length} done):`;
+        form.appendChild(label);
+
+        const checkboxes = [];
+        for (const issue of doneIssues) {
+          const row = document.createElement('div');
+          row.className = 'review-item-row';
+          const cb = document.createElement('input');
+          cb.type = 'checkbox';
+          cb.checked = true;
+          cb.dataset.issueId = issue.id;
+          checkboxes.push(cb);
+          const text = document.createElement('span');
+          text.textContent = `[${issue.priority || 'P2'}] ${issue.title}`;
+          row.appendChild(cb);
+          row.appendChild(text);
+          form.appendChild(row);
+        }
+
+        const bumpRow = document.createElement('div');
+        bumpRow.className = 'sprint-form-row';
+        bumpRow.style.marginTop = '6px';
+        const bumpLabel = document.createElement('label');
+        bumpLabel.style.cssText = 'color:var(--text-dim);font-size:calc(5px + var(--font-boost))';
+        bumpLabel.textContent = 'Bump:';
+        const bumpSel = document.createElement('select');
+        bumpSel.className = 'form-input';
+        bumpSel.style.width = 'auto';
+        bumpSel.innerHTML = '<option value="patch">Patch</option><option value="minor">Minor</option><option value="major">Major</option>';
+        bumpRow.appendChild(bumpLabel);
+        bumpRow.appendChild(bumpSel);
+
+        const releaseBtn = document.createElement('button');
+        releaseBtn.className = 'btn-small';
+        releaseBtn.textContent = 'Release';
+        releaseBtn.addEventListener('click', async () => {
+          const selectedIds = checkboxes.filter(cb => cb.checked).map(cb => cb.dataset.issueId);
+          if (!selectedIds.length) { this._showToast('Select at least one issue', 'warning'); return; }
+          try {
+            const r = await fetch(`/api/product/${encodeURIComponent(slug)}/release`, {
+              method: 'POST',
+              headers: { 'Content-Type': 'application/json' },
+              body: JSON.stringify({ resolved_issue_ids: selectedIds, bump: bumpSel.value }),
+            });
+            if (!r.ok) { const err = await r.json(); throw new Error(err.detail || r.statusText); }
+            const result = await r.json();
+            this._showToast(`Released v${result.version}`, 'success');
+            this._openProductDetail(slug);
+          } catch (err) { this._showToast(`Release failed: ${err.message}`, 'error'); }
+        });
+        bumpRow.appendChild(releaseBtn);
+
+        const cancelBtn = document.createElement('button');
+        cancelBtn.className = 'kr-remove-btn';
+        cancelBtn.innerHTML = '&times;';
+        cancelBtn.addEventListener('click', () => form.remove());
+        bumpRow.appendChild(cancelBtn);
+        form.appendChild(bumpRow);
+      });
+
+    container.appendChild(form);
+  }
+
   _doUpdateProjectsPanel() {
     const panel = document.getElementById('projects-panel-list');
     if (!panel) return;

package/frontend/style.css

@@ -6786,3 +6786,53 @@ body.resize-dragging {
   border-color: #ff4444;
   color: #ff4444;
 }
+
+/* Review Cards */
+.review-card {
+  background: rgba(0,255,255,0.03);
+  border: 1px solid rgba(0,255,255,0.1);
+  border-radius: 4px;
+  padding: 8px 10px;
+  margin-bottom: 6px;
+}
+.review-card.review-completed {
+  opacity: 0.7;
+  border-color: rgba(0,255,0,0.15);
+}
+.review-card-header {
+  display: flex;
+  gap: 8px;
+  align-items: center;
+  margin-bottom: 6px;
+  font-size: calc(5px + var(--font-boost));
+}
+.review-trigger {
+  color: var(--accent);
+  font-weight: bold;
+  text-transform: uppercase;
+  font-size: calc(5px + var(--font-boost));
+}
+.review-date { color: var(--text-dim); }
+.review-owner { color: var(--text-dim); }
+.review-items { margin: 4px 0; }
+.review-item-row {
+  display: flex;
+  align-items: center;
+  gap: 6px;
+  padding: 2px 0;
+  font-size: calc(6px + var(--font-boost));
+}
+.review-item-row input[type="checkbox"] {
+  width: 14px;
+  height: 14px;
+  accent-color: var(--accent);
+}
+.review-item-checked {
+  text-decoration: line-through;
+  color: var(--text-dim);
+}
+
+/* Release Form */
+.release-form {
+  margin-top: 8px;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@1mancompany/onemancompany",
-  "version": "0.7.32",
+  "version": "0.7.39",
   "description": "The AI Operating System for One-Person Companies",
   "bin": {
     "onemancompany": "bin/cli.js"

package/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "onemancompany"
-version = "0.7.32"
+version = "0.7.39"
 description = "A one-man company simulation with pixel art visualization and LangChain AI agents"
 requires-python = ">=3.12"
 dependencies = [

package/src/onemancompany/agents/product_tools.py

@@ -691,6 +691,58 @@ def version_management_tool(
         return f"Error: {e}"
 
 
+@tool
+async def update_product_tool(
+    product_slug: str,
+    name: str = "",
+    description: str = "",
+    objective: str = "",
+) -> str:
+    """Update a product's name, description, or objective.
+
+    Args:
+        product_slug: Product slug identifier
+        name: New product name (leave empty to keep current)
+        description: New description (leave empty to keep current)
+        objective: New objective (leave empty to keep current)
+    """
+    fields: dict = {}
+    if name:
+        fields["name"] = name
+    if description:
+        fields["description"] = description
+    if objective:
+        fields["objective"] = objective
+    if not fields:
+        return "Error: no fields to update. Provide name, description, or objective."
+    try:
+        result = prod.update_product(product_slug, **fields)
+        if result is None:
+            return f"Error: product '{product_slug}' not found"
+        return f"Updated product '{product_slug}': {', '.join(fields.keys())}"
+    except (ValueError, FileNotFoundError) as e:
+        return f"Error: {e}"
+
+
+@tool
+async def delete_product_tool(product_slug: str) -> str:
+    """Delete a product and all its issues, versions, and linked projects.
+
+    Args:
+        product_slug: Product slug identifier
+    """
+    try:
+        summary = prod.delete_product(product_slug)
+        return (
+            f"Deleted product '{product_slug}'. "
+            f"Removed {summary.get('issues', 0)} issues, "
+            f"{summary.get('versions', 0)} versions, "
+            f"{summary.get('projects', 0)} linked projects."
+        )
+    except (ValueError, FileNotFoundError) as e:
+        return f"Error: {e}"
+
+
 # ---------------------------------------------------------------------------
 # Export
 # ---------------------------------------------------------------------------
@@ -716,4 +768,6 @@ PRODUCT_TOOLS = [
     delete_sprint_tool,
     sprint_analytics_tool,
     version_management_tool,
+    update_product_tool,
+    delete_product_tool,
 ]

package/src/onemancompany/api/routes.py

@@ -7141,6 +7141,18 @@ async def api_update_kr(slug: str, kr_id: str, request: Request) -> dict:
     return result
 
 
+@router.delete("/api/product/{slug}/kr/{kr_id}")
+async def api_delete_kr(slug: str, kr_id: str) -> dict:
+    """Delete a key result from a product."""
+    from onemancompany.core import product as prod
+
+    try:
+        prod.delete_key_result(slug, kr_id)
+    except ValueError as exc:
+        raise HTTPException(status_code=404, detail=str(exc))
+    return {"ok": True}
+
+
 # ── Issues ──────────────────────────────────────────────────────────────────
 
 
@@ -7483,6 +7495,13 @@ async def api_create_sprint(slug: str, request: Request) -> dict:
         )
     except ValueError as exc:
         raise HTTPException(status_code=404, detail=str(exc))
+    await event_bus.publish(
+        CompanyEvent(
+            type=EventType.SPRINT_CREATED,
+            payload={"product_slug": slug, "sprint_id": result["id"]},
+            agent=SYSTEM_AGENT,
+        )
+    )
     return result
 
 
@@ -7531,6 +7550,13 @@ async def api_close_sprint(slug: str, sprint_id: str) -> dict:
         result = prod.close_sprint(slug, sprint_id)
     except ValueError as exc:
         raise HTTPException(status_code=400, detail=str(exc))
+    await event_bus.publish(
+        CompanyEvent(
+            type=EventType.SPRINT_CLOSED,
+            payload={"product_slug": slug, "sprint_id": sprint_id},
+            agent=SYSTEM_AGENT,
+        )
+    )
     return result
 
 
@@ -7543,6 +7569,13 @@ async def api_start_sprint(slug: str, sprint_id: str) -> dict:
         result = prod.start_sprint(slug, sprint_id)
     except ValueError as exc:
         raise HTTPException(status_code=400, detail=str(exc))
+    await event_bus.publish(
+        CompanyEvent(
+            type=EventType.SPRINT_STARTED,
+            payload={"product_slug": slug, "sprint_id": sprint_id},
+            agent=SYSTEM_AGENT,
+        )
+    )
     return result
 
 
@@ -7649,6 +7682,13 @@ async def api_create_review(slug: str, request: Request) -> dict:
         trigger_ref=trigger_ref,
         owner=owner,
     )
+    await event_bus.publish(
+        CompanyEvent(
+            type=EventType.REVIEW_CREATED,
+            payload={"product_slug": slug, "review_id": review["id"]},
+            agent=SYSTEM_AGENT,
+        )
+    )
     return review
 
 
@@ -7691,9 +7731,17 @@ async def api_complete_review(slug: str, review_id: str) -> dict:
     from onemancompany.core import product as prod
 
     try:
-        return prod.complete_review(slug, review_id)
+        result = prod.complete_review(slug, review_id)
     except ValueError as exc:
         raise HTTPException(status_code=400, detail=str(exc))
+    await event_bus.publish(
+        CompanyEvent(
+            type=EventType.REVIEW_COMPLETED,
+            payload={"product_slug": slug, "review_id": review_id},
+            agent=SYSTEM_AGENT,
+        )
+    )
+    return result
 
 
 # ---------------------------------------------------------------------------

package/src/onemancompany/core/models.py

@@ -163,6 +163,7 @@ class EventType(str, Enum):
     KR_UPDATED = "kr_updated"
     VERSION_RELEASED = "version_released"
     SPRINT_CREATED = "sprint_created"
+    SPRINT_STARTED = "sprint_started"
     SPRINT_CLOSED = "sprint_closed"
     REVIEW_CREATED = "review_created"
     REVIEW_COMPLETED = "review_completed"

package/src/onemancompany/core/product.py

@@ -37,6 +37,12 @@ from onemancompany.core.models import (
 )
 from onemancompany.core.store import _read_yaml, _write_yaml, mark_dirty
 
+# ---------------------------------------------------------------------------
+# Configurable constants
+# ---------------------------------------------------------------------------
+
+HISTORY_MAX_ENTRIES: int = 100
+
 # ---------------------------------------------------------------------------
 # Per-slug threading locks (same pattern as project_archive.py)
 # ---------------------------------------------------------------------------
@@ -230,6 +236,24 @@ def update_kr_progress(slug: str, kr_id: str, *, current: float) -> dict:
     raise ValueError(f"KR '{kr_id}' not found in product '{slug}'")
 
 
+def delete_key_result(slug: str, kr_id: str) -> None:
+    """Delete a key result from a product. Raises ValueError if not found."""
+    with _get_slug_lock(slug):
+        path = _product_yaml_path(slug)
+        data = _read_yaml(path)
+        if not data:
+            raise ValueError(f"Product '{slug}' not found")
+        krs = data.get("key_results", [])
+        original_len = len(krs)
+        data["key_results"] = [kr for kr in krs if kr["id"] != kr_id]
+        if len(data["key_results"]) == original_len:
+            raise ValueError(f"KR '{kr_id}' not found in product '{slug}'")
+        data["updated_at"] = datetime.now().isoformat()
+        _write_yaml(path, data)
+    mark_dirty(DirtyCategory.PRODUCTS)
+    logger.debug("Deleted KR {} from product {}", kr_id, slug)
+
+
 def update_kr_fields(slug: str, kr_id: str, **fields) -> dict:
     """Update arbitrary fields on a key result. Returns updated KR dict.
 
@@ -269,8 +293,8 @@ def _append_history(data: dict, field: str, old_value, new_value, changed_by: st
         "new_value": str(new_value) if new_value is not None else None,
         "changed_by": changed_by,
     })
-    if len(data["history"]) > 100:
-        data["history"] = data["history"][-100:]
+    if len(data["history"]) > HISTORY_MAX_ENTRIES:
+        data["history"] = data["history"][-HISTORY_MAX_ENTRIES:]
 
 
 def create_issue(
@@ -355,8 +379,12 @@ def list_issues(
     priority: IssuePriority | None = None,
     labels: list[str] | None = None,
     sprint: str | None = None,
+    assignee_id: str | None = None,
 ) -> list[dict]:
-    """List issues for a product, optionally filtered."""
+    """List issues for a product, optionally filtered.
+
+    assignee_id: filter by assignee. Empty string "" means unassigned.
+    """
     issues_path = _issues_dir(slug)
     if not issues_path.exists():
         return []
@@ -378,12 +406,24 @@ def list_issues(
                 continue
         if sprint is not None and data.get("sprint") != sprint:
             continue
+        if assignee_id is not None:
+            issue_assignee = data.get("assignee_id") or ""
+            if assignee_id == "":
+                # Filter for unassigned
+                if issue_assignee:
+                    continue
+            elif issue_assignee != assignee_id:
+                continue
         results.append(data)
     return results
 
 
-def update_issue(slug: str, issue_id: str, **fields) -> dict:
-    """Update issue fields. Returns updated dict. Raises ValueError if not found."""
+def update_issue(slug: str, issue_id: str, *, _skip_transition_check: bool = False, **fields) -> dict:
+    """Update issue fields. Returns updated dict. Raises ValueError if not found.
+
+    _skip_transition_check: internal flag for system-derived status updates
+    that may jump non-adjacent states (e.g. sync_issue_statuses).
+    """
     with _get_slug_lock(slug):
         path = _issues_dir(slug) / f"{issue_id}.yaml"
         data = _read_yaml(path)
@@ -391,7 +431,7 @@ def update_issue(slug: str, issue_id: str, **fields) -> dict:
             raise ValueError(f"Issue '{issue_id}' not found in product '{slug}'")
         # Validate status transition if status is being changed
         new_status = fields.get("status")
-        if new_status is not None:
+        if new_status is not None and not _skip_transition_check:
             current_status = data.get("status", IssueStatus.BACKLOG.value)
             if new_status != current_status:
                 _validate_status_transition(current_status, new_status)
@@ -959,11 +999,11 @@ def release_version(
         product["current_version"] = new_version
         _write_yaml(_product_yaml_path(product_slug), product)
 
-    # Mark resolved issues as released
+    # Mark resolved issues as released (bypass validation — release is a system operation)
     for issue_id in resolved_issue_ids:
         issue = load_issue(product_slug, issue_id)
         if issue and issue.get("status") != IssueStatus.RELEASED.value:
-            update_issue(product_slug, issue_id, status=IssueStatus.RELEASED.value)
+            update_issue(product_slug, issue_id, _skip_transition_check=True, status=IssueStatus.RELEASED.value)
 
     mark_dirty(DirtyCategory.PRODUCTS)
     logger.info("[VERSION] Released {} for product '{}'", new_version, product_slug)
@@ -994,12 +1034,14 @@ def build_product_context(product_slug: str) -> str:
             unit = kr.get("unit", "")
             suffix = f" {unit}" if unit else ""
             parts.append(f"  - {kr['title']}: {current}/{target}{suffix} ({pct:.0f}%)")
-    issues = list_issues(product_slug, status=IssueStatus.BACKLOG)
+    _terminal = {IssueStatus.DONE.value, IssueStatus.RELEASED.value}
+    issues = [i for i in list_issues(product_slug) if i.get("status") not in _terminal]
     issues.sort(key=lambda i: i.get("priority", "P3"))
     if issues:
         parts.append(f"\nActive Issues ({len(issues)}):")
         for issue in issues[:10]:
-            parts.append(f"  - [{issue['priority']}] {issue['title']} ({issue['id']})")
+            status_tag = issue.get("status", "backlog")
+            parts.append(f"  - [{issue['priority']}][{status_tag}] {issue['title']} ({issue['id']})")
         if len(issues) > 10:
             parts.append(f"  ... and {len(issues) - 10} more")
     parts.append("=== End Product Context ===")
@@ -1270,7 +1312,7 @@ def sync_issue_statuses(slug: str) -> list[dict]:
         current = issue.get("status", IssueStatus.BACKLOG.value)
 
         if derived.value != current:
-            update_issue(slug, issue["id"], status=derived.value)
+            update_issue(slug, issue["id"], _skip_transition_check=True, status=derived.value)
             changed.append({"issue_id": issue["id"], "old": current, "new": derived.value})
             logger.debug("[PRODUCT] Issue {} status derived: {} → {}", issue["id"], current, derived.value)
 

package/src/onemancompany/core/product_triggers.py

@@ -23,6 +23,16 @@ from onemancompany.core.system_cron import system_cron
 # Priorities that auto-trigger project creation
 _AUTO_PROJECT_PRIORITIES = {IssuePriority.P0.value, IssuePriority.P1.value}
 
+# ---------------------------------------------------------------------------
+# Configurable thresholds (B4 audit: extracted from inline magic numbers)
+# ---------------------------------------------------------------------------
+
+KR_LAGGING_THRESHOLD: int = 50          # KR progress % below which it's "lagging"
+MAX_ACTIVE_PROJECTS: int = 3            # Max concurrent active projects per product
+BACKLOG_GROOMING_THRESHOLD: int = 5     # P2/P3 unscheduled issues before grooming nudge
+STALE_REVIEW_HOURS: int = 24            # Hours before an open review is considered stale
+BLOCKED_DAYS_THRESHOLD: int = 7         # Days before a blocked issue is flagged
+UNHANDLED_BACKLOG_THRESHOLD: int = 2    # Unhandled backlog issues before alert
 
 # ---------------------------------------------------------------------------
 # Trigger handlers
@@ -332,7 +342,7 @@ async def check_kr_progress(product_slug: str) -> list[dict]:
         if target <= 0:
             continue
         progress_pct = current / target * 100
-        if progress_pct >= 50:
+        if progress_pct >= KR_LAGGING_THRESHOLD:
             continue
 
         # Check if an open issue already exists for this KR
@@ -415,7 +425,7 @@ async def run_product_check(product_slug: str) -> dict:
 
         # High priority + no active project → create project
         if priority in _AUTO_PROJECT_PRIORITIES and not linked:
-            if len(active_for_product) >= 3:
+            if len(active_for_product) >= MAX_ACTIVE_PROJECTS:
                 logger.debug("[PRODUCT_CHECK] Skipping project for issue {} — 3+ active projects", issue["id"])
                 continue
             project_id = await _create_project_for_issue(product_slug, issue)
@@ -430,7 +440,7 @@ async def run_product_check(product_slug: str) -> dict:
 
         # Has assignee but no project → create project
         elif issue.get("assignee_id") and not linked:
-            if len(active_for_product) >= 3:
+            if len(active_for_product) >= MAX_ACTIVE_PROJECTS:
                 continue
             project_id = await _create_project_for_issue(product_slug, issue)
             if project_id:
@@ -484,34 +494,31 @@ async def run_product_check(product_slug: str) -> dict:
             logger.debug("[PRODUCT_CHECK] Invalid end_date '{}' on sprint {}", end_date_str, active_sprint.get("id"))
 
     # --- Step 4: Backlog grooming reminder ---
-    _BACKLOG_GROOMING_THRESHOLD = 5
     unscheduled_low = [
         i for i in all_issues
         if i.get("priority") in (IssuePriority.P2.value, IssuePriority.P3.value)
         and not i.get("sprint")
         and i.get("status") not in (IssueStatus.DONE.value, IssueStatus.RELEASED.value)
     ]
-    if len(unscheduled_low) >= _BACKLOG_GROOMING_THRESHOLD:
+    if len(unscheduled_low) >= BACKLOG_GROOMING_THRESHOLD:
         actions_taken.append(f"{len(unscheduled_low)} P2/P3 issues unscheduled — backlog grooming needed")
 
-    # --- Step 5: Stale review check (open > 24h) ---
+    # --- Step 5: Stale review check ---
     from datetime import datetime as _datetime, timedelta as _timedelta
 
     open_reviews = prod.list_reviews(product_slug, status="open")
-    _STALE_REVIEW_HOURS = 24
     stale_reviews = []
     for rev in open_reviews:
         try:
             created = _datetime.fromisoformat(rev.get("created_at", ""))
-            if _datetime.now() - created > _timedelta(hours=_STALE_REVIEW_HOURS):
+            if _datetime.now() - created > _timedelta(hours=STALE_REVIEW_HOURS):
                 stale_reviews.append(rev)
         except (ValueError, TypeError):
             logger.debug("[PRODUCT_CHECK] Invalid created_at on review {}", rev.get("id"))
     if stale_reviews:
-        actions_taken.append(f"{len(stale_reviews)} stale review(s) open > {_STALE_REVIEW_HOURS}h")
+        actions_taken.append(f"{len(stale_reviews)} stale review(s) open > {STALE_REVIEW_HOURS}h")
 
-    # --- Step 6: Blocked issue check (blocked > 7 days) ---
-    _BLOCKED_DAYS_THRESHOLD = 7
+    # --- Step 6: Blocked issue check ---
     for issue in all_issues:
         if issue.get("status") in (IssueStatus.DONE.value, IssueStatus.RELEASED.value):
             continue
@@ -532,9 +539,9 @@ async def run_product_check(product_slug: str) -> dict:
                     oldest_blocked_at = link_created
             except (ValueError, TypeError):
                 logger.debug("[PRODUCT_CHECK] Invalid created_at on link in issue {}", issue.get("id"))
-        if oldest_blocked_at and _datetime.now() - oldest_blocked_at > _timedelta(days=_BLOCKED_DAYS_THRESHOLD):
+        if oldest_blocked_at and _datetime.now() - oldest_blocked_at > _timedelta(days=BLOCKED_DAYS_THRESHOLD):
             actions_taken.append(
-                f"Issue '{issue['title']}' blocked for >{_BLOCKED_DAYS_THRESHOLD} days"
+                f"Issue '{issue['title']}' blocked for >{BLOCKED_DAYS_THRESHOLD} days"
             )
 
     # --- Step 7: Check if owner review is needed ---
@@ -546,7 +553,7 @@ async def run_product_check(product_slug: str) -> dict:
         i for i in all_issues
         if i.get("status") == IssueStatus.BACKLOG.value and not i.get("linked_task_ids")
     ]
-    if len(unhandled_backlog) > 2:
+    if len(unhandled_backlog) > UNHANDLED_BACKLOG_THRESHOLD:
         needs_review = True
         review_reasons.append(f"{len(unhandled_backlog)} unhandled backlog issues")
 
@@ -570,7 +577,7 @@ async def run_product_check(product_slug: str) -> dict:
             logger.debug("[PRODUCT_CHECK] Invalid end_date on sprint {} for review check", active_sprint.get("id"))
 
     # Backlog grooming threshold → needs owner review
-    if len(unscheduled_low) >= _BACKLOG_GROOMING_THRESHOLD:
+    if len(unscheduled_low) >= BACKLOG_GROOMING_THRESHOLD:
         needs_review = True
         review_reasons.append(f"{len(unscheduled_low)} P2/P3 issues need sprint assignment")
 
@@ -755,6 +762,7 @@ def register_product_triggers() -> "asyncio.Task":
         EventType.ISSUE_CLOSED,
         EventType.ISSUE_ASSIGNED,
         EventType.SPRINT_CREATED,
+        EventType.SPRINT_STARTED,
         EventType.SPRINT_CLOSED,
         EventType.VERSION_RELEASED,
         EventType.REVIEW_CREATED,