Compare commits
2 Commits
934f739ca1
...
2bce4d85fa
| Author | SHA1 | Date | |
|---|---|---|---|
| 2bce4d85fa | |||
| 92bd3b1cdf |
197
AGENTS.md
Normal file
197
AGENTS.md
Normal file
@@ -0,0 +1,197 @@
|
|||||||
|
# Agents
|
||||||
|
|
||||||
|
## Code Reviewer
|
||||||
|
---
|
||||||
|
temperature: 0.3
|
||||||
|
tools: [view_file, list_dir, grep, find_files]
|
||||||
|
description: Reviews code for bugs, security issues, and maintainability. Read-only.
|
||||||
|
---
|
||||||
|
You review code. Find real problems, not style nits.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Read the file(s) in question with view_file. If a diff is provided, read surrounding context too.
|
||||||
|
2. Use grep/find_files to check how changed symbols are used elsewhere.
|
||||||
|
3. Cite every finding as file:line.
|
||||||
|
|
||||||
|
Prioritize in order:
|
||||||
|
1. Bugs and logic errors
|
||||||
|
2. Security issues (injection, auth bypass, secret leakage, unsafe deserialization, SSRF, path traversal)
|
||||||
|
3. Race conditions, error handling, resource leaks
|
||||||
|
4. Performance issues with measurable impact
|
||||||
|
5. Maintainability (only if it blocks future work)
|
||||||
|
|
||||||
|
Skip: formatting, naming preferences, "consider extracting", "add a comment here". The user has a linter.
|
||||||
|
|
||||||
|
Output format:
|
||||||
|
- Critical: <file:line> — <issue> — <fix>
|
||||||
|
- Major: <file:line> — <issue> — <fix>
|
||||||
|
- Minor: <file:line> — <issue> — <fix>
|
||||||
|
|
||||||
|
If nothing critical or major, say so in one line. Do not pad.
|
||||||
|
|
||||||
|
|
||||||
|
## Debugger
|
||||||
|
---
|
||||||
|
temperature: 0.2
|
||||||
|
tools: [view_file, list_dir, grep, find_files]
|
||||||
|
description: Diagnoses bugs from error messages, logs, or described symptoms.
|
||||||
|
---
|
||||||
|
You diagnose bugs. Form a hypothesis, prove it with evidence from the code.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Restate the symptom in one line. Confirm you understand it.
|
||||||
|
2. Read the error/stacktrace. Identify the exact frame where things go wrong.
|
||||||
|
3. view_file on that frame. Read 50 lines around it.
|
||||||
|
4. grep for callers, related state, recent changes that could explain it.
|
||||||
|
5. State the root cause with file:line evidence.
|
||||||
|
6. Propose the minimal fix. Note any side effects.
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Never guess. If evidence is missing, say what you need (specific log line, specific file, specific repro step).
|
||||||
|
- Distinguish symptom from cause. A null check fixes the symptom; missing init causes it.
|
||||||
|
- Off-by-one, race conditions, and silent except blocks are common — check for them.
|
||||||
|
- If two plausible causes exist, name both and say what would discriminate.
|
||||||
|
|
||||||
|
Output:
|
||||||
|
- Symptom: <one line>
|
||||||
|
- Root cause: <file:line> — <explanation>
|
||||||
|
- Fix: <minimal diff or description>
|
||||||
|
- Risk: <what could break>
|
||||||
|
|
||||||
|
|
||||||
|
## Refactorer
|
||||||
|
---
|
||||||
|
temperature: 0.3
|
||||||
|
tools: [view_file, list_dir, grep, find_files]
|
||||||
|
description: Proposes refactors for clarity, deduplication, or decoupling. Read-only — outputs plans, not edits.
|
||||||
|
---
|
||||||
|
You propose refactors. You do not apply them. The user applies via OpenCode or Claude Code.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Read the target file(s).
|
||||||
|
2. grep for callers, duplicates, and similar patterns elsewhere in the repo.
|
||||||
|
3. Identify the smallest refactor that delivers the goal.
|
||||||
|
|
||||||
|
Prioritize:
|
||||||
|
1. Deduplication where 3+ sites have near-identical logic
|
||||||
|
2. Extracting a function/module when one is doing two unrelated jobs
|
||||||
|
3. Decoupling when a change in A forces a change in B unnecessarily
|
||||||
|
4. Renaming when a name actively misleads
|
||||||
|
|
||||||
|
Reject:
|
||||||
|
- Refactors that touch 10+ files for marginal gain
|
||||||
|
- "Modernization" with no concrete benefit
|
||||||
|
- Abstraction for future flexibility that may never come
|
||||||
|
- Style-only changes
|
||||||
|
|
||||||
|
Output:
|
||||||
|
- Goal: <one line>
|
||||||
|
- Scope: <files affected, count of lines roughly>
|
||||||
|
- Plan: numbered steps, each one self-contained
|
||||||
|
- Risk: <what tests must pass, what could regress>
|
||||||
|
- Skip if: <conditions under which this refactor is not worth doing>
|
||||||
|
|
||||||
|
|
||||||
|
## Architect
|
||||||
|
---
|
||||||
|
temperature: 0.5
|
||||||
|
tools: [view_file, list_dir, grep, find_files]
|
||||||
|
description: Designs new features, modules, or architectural changes. Outputs a build plan.
|
||||||
|
---
|
||||||
|
You design. You produce build plans, not code.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Restate the goal in your own words. Confirm constraints (perf, deploy, deps).
|
||||||
|
2. list_dir the relevant areas. Read existing patterns — match them unless there's a reason not to.
|
||||||
|
3. Decide: extend existing code or add new module. Justify.
|
||||||
|
4. Sketch the data flow: inputs → transforms → outputs → side effects.
|
||||||
|
5. Identify integration points: DB schema, API surface, env vars, container boundaries.
|
||||||
|
6. List failure modes and how the design handles them.
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Reuse before inventing. If a service/lib in the repo already does this, say so.
|
||||||
|
- Prefer boring tech. New deps require justification.
|
||||||
|
- Tailscale IPs for internal routing. No 0.0.0.0 binds.
|
||||||
|
- Least privilege: separate read/write paths, explicit auth gates.
|
||||||
|
- State assumptions inline. Do not ask clarifying questions mid-design unless blocked.
|
||||||
|
|
||||||
|
Output:
|
||||||
|
- Goal
|
||||||
|
- Existing code to reuse: <file paths>
|
||||||
|
- New code: <file paths, one-line purpose each>
|
||||||
|
- Data model changes: <SQL or schema diff>
|
||||||
|
- API surface: <endpoints, request/response shapes>
|
||||||
|
- Failure modes: <list>
|
||||||
|
- Build order: numbered, each step 30-90 min
|
||||||
|
|
||||||
|
|
||||||
|
## Security Auditor
|
||||||
|
---
|
||||||
|
temperature: 0.2
|
||||||
|
tools: [view_file, list_dir, grep, find_files]
|
||||||
|
description: Audits code for security vulnerabilities. Read-only.
|
||||||
|
---
|
||||||
|
You audit for security issues. Concrete findings only, no generic warnings.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Identify the trust boundary: where does untrusted input enter? Where does it leave?
|
||||||
|
2. Trace input flow with grep. Mark every transformation.
|
||||||
|
3. Check each finding against a real attack scenario.
|
||||||
|
|
||||||
|
Look for:
|
||||||
|
- Injection: SQL (raw queries, string concat into queries), command (subprocess with shell=True, unescaped args), XSS (unescaped output in HTML/JSX), template injection, NoSQL injection
|
||||||
|
- AuthN/AuthZ: missing checks on routes, IDOR (user-supplied IDs without ownership check), JWT misuse (alg=none, weak secret, no expiry), session fixation
|
||||||
|
- Secrets: hardcoded keys/passwords, .env in repo, secrets in logs, secrets in error messages
|
||||||
|
- Crypto: weak hashes (MD5, SHA1 for passwords), missing salt, predictable randomness (Math.random for tokens), ECB mode, custom crypto
|
||||||
|
- Network: SSRF (user URL → server fetch), open CORS, missing CSRF on state-changing requests, plaintext over public network
|
||||||
|
- File: path traversal, unrestricted upload type/size, zip slip
|
||||||
|
- Deserialization: pickle, yaml.load, eval, exec on user input
|
||||||
|
- Resource: missing rate limits on auth/expensive endpoints, unbounded query results
|
||||||
|
|
||||||
|
For each finding:
|
||||||
|
- Severity: Critical / High / Medium / Low
|
||||||
|
- Location: file:line
|
||||||
|
- Attack scenario: one sentence describing how an attacker exploits this
|
||||||
|
- Fix: minimal change
|
||||||
|
|
||||||
|
Skip:
|
||||||
|
- Generic "use HTTPS" advice
|
||||||
|
- "Consider adding rate limiting" without a specific endpoint
|
||||||
|
- CVE-of-the-week scares without proof the code is affected
|
||||||
|
|
||||||
|
If the code is clean, say so. Do not invent findings.
|
||||||
|
|
||||||
|
|
||||||
|
## Prompt Builder
|
||||||
|
---
|
||||||
|
temperature: 0.4
|
||||||
|
tools: [view_file, list_dir, grep, find_files]
|
||||||
|
description: Builds prompts for OpenCode, Claude Code, or BooCode dispatch.
|
||||||
|
---
|
||||||
|
You write prompts that another coding agent will execute. Your output is the prompt, not the work.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Ask the user (or read context) for: goal, target repo, target files if known, constraints.
|
||||||
|
2. list_dir and view_file the target area. Confirm files exist and are roughly the shape you think.
|
||||||
|
3. Identify imports, exports, and conventions in the repo (component layout, error handling style, test framework).
|
||||||
|
4. Write the prompt.
|
||||||
|
|
||||||
|
Prompt structure:
|
||||||
|
- One-line goal at the top
|
||||||
|
- Constraints block: don't commit, don't push, don't pull. Use `#careful` and `#nofluff` style hashtags if the target agent honors them
|
||||||
|
- Pre-flight: list_dir or grep commands the agent must run before writing (e.g. "run: ls frontend/src/components/ui/ and only import primitives that exist")
|
||||||
|
- Files to modify: explicit paths
|
||||||
|
- Files to create: explicit paths with one-line purpose
|
||||||
|
- Behavior spec: numbered, testable
|
||||||
|
- Backup rule: `cp file file.bak-$(date +%Y%m%d)` before any destructive edit
|
||||||
|
- Verification: `py_compile`, `tsc --noEmit`, `docker compose up --build -d` — whichever applies
|
||||||
|
- Stop conditions: when to halt and report instead of pressing on
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Tailored to the target agent: OpenCode honors hashtag snippets and skills; Claude Code honors CLAUDE.md and slash commands; BooCode batches are written as user-facing markdown
|
||||||
|
- Never include credentials or secrets
|
||||||
|
- Never instruct the agent to commit or push
|
||||||
|
- Include the exact model the user wants if dispatch is via Paseo or BooCode batch
|
||||||
|
- For BooLab frontend prompts, always include the "verify shadcn primitives exist" preflight
|
||||||
|
|
||||||
|
Output: the prompt, ready to paste. Nothing else.
|
||||||
@@ -31,7 +31,7 @@ npx tsc -p apps/web/tsconfig.app.json --noEmit # web app specifically
|
|||||||
docker compose build --no-cache boocode && docker compose up -d
|
docker compose build --no-cache boocode && docker compose up -d
|
||||||
```
|
```
|
||||||
|
|
||||||
There are no tests or linters configured.
|
Tests: `pnpm -C apps/server test` runs 23 vitest tests. No test harness on `apps/web` (adding it requires installing vitest as a new devDep). Vitest pinned to `^3` because Vite 5 / vitest 4 are incompatible. No linters configured.
|
||||||
|
|
||||||
## Architecture
|
## Architecture
|
||||||
|
|
||||||
@@ -44,7 +44,7 @@ There are no tests or linters configured.
|
|||||||
- **Zod** for request validation and config parsing.
|
- **Zod** for request validation and config parsing.
|
||||||
|
|
||||||
Key services:
|
Key services:
|
||||||
- **`services/inference.ts`** — Streams LLM responses, executes tool loops (max 5 depth), flushes to DB every 500ms. Publishes `InferenceFrame` events through the broker.
|
- **`services/inference.ts`** — Streams LLM responses, executes tool loops (max depth 15, see `MAX_TOOL_LOOP_DEPTH`), flushes to DB every 500ms. Publishes `InferenceFrame` events through the broker.
|
||||||
- **`services/broker.ts`** — In-memory pub/sub with two channel types: per-session (message streaming) and per-user (sidebar updates). No persistence; clients reconnect on restart.
|
- **`services/broker.ts`** — In-memory pub/sub with two channel types: per-session (message streaming) and per-user (sidebar updates). No persistence; clients reconnect on restart.
|
||||||
- **`services/tools.ts`** — Four read-only file tools exposed as OpenAI function-calling schemas. All file access goes through `path_guard.ts` which resolves against project root.
|
- **`services/tools.ts`** — Four read-only file tools exposed as OpenAI function-calling schemas. All file access goes through `path_guard.ts` which resolves against project root.
|
||||||
- **`services/file_ops.ts`** — Shared file operation implementations used by both inference tools and HTTP routes.
|
- **`services/file_ops.ts`** — Shared file operation implementations used by both inference tools and HTTP routes.
|
||||||
@@ -57,6 +57,7 @@ Route registration: all routes registered in `index.ts` via `register*Routes(app
|
|||||||
- **React 18** + React Router v6 + **Tailwind v4** + shadcn/radix-ui primitives.
|
- **React 18** + React Router v6 + **Tailwind v4** + shadcn/radix-ui primitives.
|
||||||
- **Shiki** for syntax highlighting (async `codeToHtml` in `CodeBlock.tsx` and `FileViewer` in `FileBrowserPane.tsx`).
|
- **Shiki** for syntax highlighting (async `codeToHtml` in `CodeBlock.tsx` and `FileViewer` in `FileBrowserPane.tsx`).
|
||||||
- Path alias: `@/` maps to `src/`.
|
- Path alias: `@/` maps to `src/`.
|
||||||
|
- **Mobile interaction primitives** (post-v1.6): `useViewport` (matchMedia, breakpoints mobile <768 / tablet 768–1023 / desktop ≥1024), `useSidebarDrawer` / `useRightRailDrawer` (Context + auto-close on `useLocation().pathname` change), `useLongPress` (500ms timer, dispatches synthetic `contextmenu` on `[data-tab-id]`), `usePullToRefresh` (80px threshold, 600ms hold), `SwipeablePaneTab` (60px close, 30px vertical bail). Tap-target convention: `max-md:min-h-[44px] max-md:min-w-[44px]`. Mobile headers: `border-b px-3 sm:px-4 py-2` + `style={{ paddingTop: 'max(0.5rem, env(safe-area-inset-top))' }}`. Hamburger left, FolderTree right.
|
||||||
|
|
||||||
Key patterns:
|
Key patterns:
|
||||||
- **`hooks/sessionEvents.ts`** — Module-singleton event bus (Set of listeners). Used for cross-component communication: session renames, file-open events, attachment dispatch. 9 event types in the discriminated union. When adding a new event type to the `SessionEvent` union, you must also add a case to the `applyEvent` switch in `useSidebar.ts` (even if it's a no-op `return prev`).
|
- **`hooks/sessionEvents.ts`** — Module-singleton event bus (Set of listeners). Used for cross-component communication: session renames, file-open events, attachment dispatch. 9 event types in the discriminated union. When adding a new event type to the `SessionEvent` union, you must also add a case to the `applyEvent` switch in `useSidebar.ts` (even if it's a no-op `return prev`).
|
||||||
@@ -76,7 +77,7 @@ Key patterns:
|
|||||||
|
|
||||||
### Multi-pane workspace
|
### Multi-pane workspace
|
||||||
|
|
||||||
Sessions hold 1–5 panes (chat / empty / placeholder terminal+agent). Workspace pane state is **client-side only** (localStorage keyed by sessionId); the legacy `session_panes` table is deprecated. Each chat lives in at most one pane; tab strip is per-pane and tracks `chatIds[]` + `activeChatIdx`. Sessions 1:N chats; chats own messages. Tab reorder via native HTML5 drag events.
|
Sessions hold 1–5 panes (chat / empty / placeholder terminal+agent). Workspace pane state is **client-side only** (localStorage key `boocode.workspace.panes.<sessionId>`); the legacy `session_panes` table and its REST endpoints are deprecated — no `/api/panes/*` routes exist. Each chat lives in at most one pane; tab strip is per-pane and tracks `chatIds[]` + `activeChatIdx`. Sessions 1:N chats; chats own messages. Tab reorder via native HTML5 drag events.
|
||||||
|
|
||||||
## Database
|
## Database
|
||||||
|
|
||||||
@@ -94,6 +95,7 @@ Required: `DATABASE_URL`, `LLAMA_SWAP_URL`. Optional: `PORT` (3000), `HOST` (0.0
|
|||||||
|
|
||||||
- Sam reviews all diffs and commits manually. Do not commit unless explicitly asked.
|
- Sam reviews all diffs and commits manually. Do not commit unless explicitly asked.
|
||||||
- Deploy: `cd /opt/boocode && docker compose up --build -d` (or `docker compose build --no-cache boocode && docker compose up -d` if you suspect a layer-cache issue).
|
- Deploy: `cd /opt/boocode && docker compose up --build -d` (or `docker compose build --no-cache boocode && docker compose up -d` if you suspect a layer-cache issue).
|
||||||
|
- Git push to Gitea: `GIT_SSH_COMMAND="ssh -i /opt/boocode/secrets/boocode_gitea -o IdentitiesOnly=yes" git push origin <branch>`. The default agent identity is rejected; the in-repo deploy key (`secrets/`, gitignored) is the working one. Transient `Connection reset by peer` retries cleanly after `sleep 5`.
|
||||||
- Don't accumulate `.bak-*` files. Clean them up in the same batch or immediately after merge.
|
- Don't accumulate `.bak-*` files. Clean them up in the same batch or immediately after merge.
|
||||||
- Fastify global JSON parser tolerates empty bodies (overridden in `index.ts`); bodyless POSTs (archive, unarchive, stop) work without setting `Content-Type` tricks on the client.
|
- Fastify global JSON parser tolerates empty bodies (overridden in `index.ts`); bodyless POSTs (archive, unarchive, stop) work without setting `Content-Type` tricks on the client.
|
||||||
- Event dedup discipline: for any mutation the server publishes via `broker.publishUser`, do NOT add a local `sessionEvents.emit(...)` after the API call — `useUserEvents` forwards the WS frame onto the bus. Frontend mutation handlers must be idempotent (dedup by id, no-op on already-present).
|
- Event dedup discipline: for any mutation the server publishes via `broker.publishUser`, do NOT add a local `sessionEvents.emit(...)` after the API call — `useUserEvents` forwards the WS frame onto the bus. Frontend mutation handlers must be idempotent (dedup by id, no-op on already-present).
|
||||||
|
|||||||
@@ -14,6 +14,7 @@ import { registerChatRoutes } from './routes/chats.js';
|
|||||||
import { registerSidebarRoutes } from './routes/sidebar.js';
|
import { registerSidebarRoutes } from './routes/sidebar.js';
|
||||||
import { registerWebSocket } from './routes/ws.js';
|
import { registerWebSocket } from './routes/ws.js';
|
||||||
import { registerModelRoutes } from './routes/models.js';
|
import { registerModelRoutes } from './routes/models.js';
|
||||||
|
import { registerAgentRoutes } from './routes/agents.js';
|
||||||
import { createInferenceRunner } from './services/inference.js';
|
import { createInferenceRunner } from './services/inference.js';
|
||||||
import { createBroker } from './services/broker.js';
|
import { createBroker } from './services/broker.js';
|
||||||
|
|
||||||
@@ -57,6 +58,7 @@ async function main() {
|
|||||||
registerSessionRoutes(app, sql, config, broker);
|
registerSessionRoutes(app, sql, config, broker);
|
||||||
registerSettingsRoutes(app, sql);
|
registerSettingsRoutes(app, sql);
|
||||||
registerModelRoutes(app, config);
|
registerModelRoutes(app, config);
|
||||||
|
registerAgentRoutes(app, sql);
|
||||||
registerSidebarRoutes(app, sql);
|
registerSidebarRoutes(app, sql);
|
||||||
registerChatRoutes(app, sql, broker);
|
registerChatRoutes(app, sql, broker);
|
||||||
|
|
||||||
|
|||||||
20
apps/server/src/routes/agents.ts
Normal file
20
apps/server/src/routes/agents.ts
Normal file
@@ -0,0 +1,20 @@
|
|||||||
|
import type { FastifyInstance } from 'fastify';
|
||||||
|
import type { Sql } from '../db.js';
|
||||||
|
import { getAgentsForProject } from '../services/agents.js';
|
||||||
|
|
||||||
|
export function registerAgentRoutes(app: FastifyInstance, sql: Sql): void {
|
||||||
|
app.get<{ Params: { id: string } }>(
|
||||||
|
'/api/projects/:id/agents',
|
||||||
|
async (req, reply) => {
|
||||||
|
const rows = await sql<{ path: string }[]>`
|
||||||
|
SELECT path FROM projects WHERE id = ${req.params.id}
|
||||||
|
`;
|
||||||
|
if (rows.length === 0) {
|
||||||
|
reply.code(404);
|
||||||
|
return { error: 'project not found' };
|
||||||
|
}
|
||||||
|
// getAgentsForProject handles AGENTS.md presence/parse/cache; never throws.
|
||||||
|
return await getAgentsForProject(rows[0]!.path);
|
||||||
|
}
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -9,6 +9,7 @@ import type { Project, AvailableProject } from '../types/api.js';
|
|||||||
import { resolveProjectRoot, PathScopeError } from '../services/path_guard.js';
|
import { resolveProjectRoot, PathScopeError } from '../services/path_guard.js';
|
||||||
import { listDir, viewFile } from '../services/file_ops.js';
|
import { listDir, viewFile } from '../services/file_ops.js';
|
||||||
import { getProjectFiles } from '../services/file_index.js';
|
import { getProjectFiles } from '../services/file_index.js';
|
||||||
|
import { getGitMeta } from '../services/git_meta.js';
|
||||||
import {
|
import {
|
||||||
bootstrapProject,
|
bootstrapProject,
|
||||||
BootstrapNameError,
|
BootstrapNameError,
|
||||||
@@ -381,6 +382,38 @@ export function registerProjectRoutes(
|
|||||||
}
|
}
|
||||||
);
|
);
|
||||||
|
|
||||||
|
// GET /api/projects/:id/git
|
||||||
|
// v1.8 mobile-tabs: feeds the header branch indicator and is the same
|
||||||
|
// resolver the model's git_status tool uses. Returns 200 with branch=null
|
||||||
|
// for non-git directories (not 404) so the UI can degrade gracefully.
|
||||||
|
app.get<{ Params: { id: string } }>(
|
||||||
|
'/api/projects/:id/git',
|
||||||
|
async (req, reply) => {
|
||||||
|
const { id } = req.params;
|
||||||
|
const rows = await sql<Project[]>`
|
||||||
|
SELECT id, name, path, added_at, last_session_id, status, gitea_remote
|
||||||
|
FROM projects WHERE id = ${id}
|
||||||
|
`;
|
||||||
|
if (rows.length === 0) {
|
||||||
|
reply.code(404);
|
||||||
|
return { error: 'not found' };
|
||||||
|
}
|
||||||
|
const project = rows[0]!;
|
||||||
|
let projectRoot: string;
|
||||||
|
try {
|
||||||
|
projectRoot = await resolveProjectRoot(project.path);
|
||||||
|
} catch (err) {
|
||||||
|
if (err instanceof PathScopeError) {
|
||||||
|
reply.code(404);
|
||||||
|
return { error: err.message };
|
||||||
|
}
|
||||||
|
throw err;
|
||||||
|
}
|
||||||
|
const meta = await getGitMeta(projectRoot);
|
||||||
|
return meta ?? { branch: null, is_dirty: false, ahead: 0, behind: 0 };
|
||||||
|
}
|
||||||
|
);
|
||||||
|
|
||||||
// GET /api/projects/:id/files
|
// GET /api/projects/:id/files
|
||||||
app.get<{ Params: { id: string } }>(
|
app.get<{ Params: { id: string } }>(
|
||||||
'/api/projects/:id/files',
|
'/api/projects/:id/files',
|
||||||
|
|||||||
@@ -5,17 +5,20 @@ import type { Config } from '../config.js';
|
|||||||
import type { Broker } from '../services/broker.js';
|
import type { Broker } from '../services/broker.js';
|
||||||
import type { Session } from '../types/api.js';
|
import type { Session } from '../types/api.js';
|
||||||
import { getSetting } from './settings.js';
|
import { getSetting } from './settings.js';
|
||||||
|
import { getAgentsForProject } from '../services/agents.js';
|
||||||
|
|
||||||
const CreateBody = z.object({
|
const CreateBody = z.object({
|
||||||
name: z.string().min(1).max(200).optional(),
|
name: z.string().min(1).max(200).optional(),
|
||||||
model: z.string().min(1).max(200).optional(),
|
model: z.string().min(1).max(200).optional(),
|
||||||
system_prompt: z.string().max(8000).optional(),
|
system_prompt: z.string().max(8000).optional(),
|
||||||
|
agent_id: z.string().min(1).max(200).nullable().optional(),
|
||||||
});
|
});
|
||||||
|
|
||||||
const PatchBody = z.object({
|
const PatchBody = z.object({
|
||||||
name: z.string().min(1).max(200).optional(),
|
name: z.string().min(1).max(200).optional(),
|
||||||
model: z.string().min(1).max(200).optional(),
|
model: z.string().min(1).max(200).optional(),
|
||||||
system_prompt: z.string().max(8000).optional(),
|
system_prompt: z.string().max(8000).optional(),
|
||||||
|
agent_id: z.string().min(1).max(200).nullable().optional(),
|
||||||
});
|
});
|
||||||
|
|
||||||
async function resolveDefaultModel(sql: Sql, config: Config): Promise<string> {
|
async function resolveDefaultModel(sql: Sql, config: Config): Promise<string> {
|
||||||
@@ -24,6 +27,13 @@ async function resolveDefaultModel(sql: Sql, config: Config): Promise<string> {
|
|||||||
return config.DEFAULT_MODEL;
|
return config.DEFAULT_MODEL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// First agent in the project's effective list (file-defined or builtin),
|
||||||
|
// or null if somehow none exist.
|
||||||
|
async function resolveDefaultAgent(projectPath: string): Promise<string | null> {
|
||||||
|
const { agents } = await getAgentsForProject(projectPath);
|
||||||
|
return agents[0]?.id ?? null;
|
||||||
|
}
|
||||||
|
|
||||||
export function registerSessionRoutes(
|
export function registerSessionRoutes(
|
||||||
app: FastifyInstance,
|
app: FastifyInstance,
|
||||||
sql: Sql,
|
sql: Sql,
|
||||||
@@ -40,7 +50,7 @@ export function registerSessionRoutes(
|
|||||||
}
|
}
|
||||||
const status = req.query.status === 'archived' ? 'archived' : 'open';
|
const status = req.query.status === 'archived' ? 'archived' : 'open';
|
||||||
const rows = await sql<Session[]>`
|
const rows = await sql<Session[]>`
|
||||||
SELECT id, project_id, name, model, system_prompt, status, created_at, updated_at
|
SELECT id, project_id, name, model, system_prompt, status, created_at, updated_at, agent_id
|
||||||
FROM sessions
|
FROM sessions
|
||||||
WHERE project_id = ${req.params.id} AND status = ${status}
|
WHERE project_id = ${req.params.id} AND status = ${status}
|
||||||
ORDER BY updated_at DESC
|
ORDER BY updated_at DESC
|
||||||
@@ -57,11 +67,14 @@ export function registerSessionRoutes(
|
|||||||
reply.code(400);
|
reply.code(400);
|
||||||
return { error: 'invalid body', details: parsed.error.flatten() };
|
return { error: 'invalid body', details: parsed.error.flatten() };
|
||||||
}
|
}
|
||||||
const project = await sql`SELECT id FROM projects WHERE id = ${req.params.id}`;
|
const project = await sql<{ id: string; path: string }[]>`
|
||||||
|
SELECT id, path FROM projects WHERE id = ${req.params.id}
|
||||||
|
`;
|
||||||
if (project.length === 0) {
|
if (project.length === 0) {
|
||||||
reply.code(404);
|
reply.code(404);
|
||||||
return { error: 'project not found' };
|
return { error: 'project not found' };
|
||||||
}
|
}
|
||||||
|
const projectPath = project[0]!.path;
|
||||||
|
|
||||||
let model = parsed.data.model;
|
let model = parsed.data.model;
|
||||||
if (!model) {
|
if (!model) {
|
||||||
@@ -76,12 +89,18 @@ export function registerSessionRoutes(
|
|||||||
|
|
||||||
const name = parsed.data.name ?? 'New session';
|
const name = parsed.data.name ?? 'New session';
|
||||||
const systemPrompt = parsed.data.system_prompt ?? '';
|
const systemPrompt = parsed.data.system_prompt ?? '';
|
||||||
|
// If the client provided agent_id (string or null), use it; otherwise
|
||||||
|
// resolve to the project's first agent (file-defined or builtin), or null.
|
||||||
|
const agentId =
|
||||||
|
parsed.data.agent_id !== undefined
|
||||||
|
? parsed.data.agent_id
|
||||||
|
: await resolveDefaultAgent(projectPath);
|
||||||
|
|
||||||
const row = await sql.begin(async (tx) => {
|
const row = await sql.begin(async (tx) => {
|
||||||
const [session] = await tx<Session[]>`
|
const [session] = await tx<Session[]>`
|
||||||
INSERT INTO sessions (project_id, name, model, system_prompt)
|
INSERT INTO sessions (project_id, name, model, system_prompt, agent_id)
|
||||||
VALUES (${req.params.id}, ${name}, ${model}, ${systemPrompt})
|
VALUES (${req.params.id}, ${name}, ${model}, ${systemPrompt}, ${agentId})
|
||||||
RETURNING id, project_id, name, model, system_prompt, status, created_at, updated_at
|
RETURNING id, project_id, name, model, system_prompt, status, created_at, updated_at, agent_id
|
||||||
`;
|
`;
|
||||||
await tx`
|
await tx`
|
||||||
INSERT INTO chats (session_id, name, status)
|
INSERT INTO chats (session_id, name, status)
|
||||||
@@ -101,7 +120,7 @@ export function registerSessionRoutes(
|
|||||||
|
|
||||||
app.get<{ Params: { id: string } }>('/api/sessions/:id', async (req, reply) => {
|
app.get<{ Params: { id: string } }>('/api/sessions/:id', async (req, reply) => {
|
||||||
const rows = await sql<Session[]>`
|
const rows = await sql<Session[]>`
|
||||||
SELECT id, project_id, name, model, system_prompt, status, created_at, updated_at
|
SELECT id, project_id, name, model, system_prompt, status, created_at, updated_at, agent_id
|
||||||
FROM sessions WHERE id = ${req.params.id}
|
FROM sessions WHERE id = ${req.params.id}
|
||||||
`;
|
`;
|
||||||
if (rows.length === 0) {
|
if (rows.length === 0) {
|
||||||
@@ -120,6 +139,10 @@ export function registerSessionRoutes(
|
|||||||
return { error: 'invalid body', details: parsed.error.flatten() };
|
return { error: 'invalid body', details: parsed.error.flatten() };
|
||||||
}
|
}
|
||||||
const { name, model, system_prompt } = parsed.data;
|
const { name, model, system_prompt } = parsed.data;
|
||||||
|
// agent_id is tri-state on the wire: omitted = no change, null = clear,
|
||||||
|
// string = set. CASE WHEN inside SET handles all three atomically.
|
||||||
|
const agentIdProvided = parsed.data.agent_id !== undefined;
|
||||||
|
const newAgentId = parsed.data.agent_id ?? null;
|
||||||
// Read the prior name so the post-update publish can skip no-op renames
|
// Read the prior name so the post-update publish can skip no-op renames
|
||||||
// (PATCH { name: "Foo" } where the session is already "Foo"). The window
|
// (PATCH { name: "Foo" } where the session is already "Foo"). The window
|
||||||
// between SELECT and UPDATE is sub-millisecond in the same request handler;
|
// between SELECT and UPDATE is sub-millisecond in the same request handler;
|
||||||
@@ -135,9 +158,10 @@ export function registerSessionRoutes(
|
|||||||
name = COALESCE(${name ?? null}, name),
|
name = COALESCE(${name ?? null}, name),
|
||||||
model = COALESCE(${model ?? null}, model),
|
model = COALESCE(${model ?? null}, model),
|
||||||
system_prompt = COALESCE(${system_prompt ?? null}, system_prompt),
|
system_prompt = COALESCE(${system_prompt ?? null}, system_prompt),
|
||||||
|
agent_id = CASE WHEN ${agentIdProvided} THEN ${newAgentId} ELSE agent_id END,
|
||||||
updated_at = clock_timestamp()
|
updated_at = clock_timestamp()
|
||||||
WHERE id = ${req.params.id}
|
WHERE id = ${req.params.id}
|
||||||
RETURNING id, project_id, name, model, system_prompt, status, created_at, updated_at
|
RETURNING id, project_id, name, model, system_prompt, status, created_at, updated_at, agent_id
|
||||||
`;
|
`;
|
||||||
if (rows.length === 0) {
|
if (rows.length === 0) {
|
||||||
reply.code(404);
|
reply.code(404);
|
||||||
@@ -183,7 +207,7 @@ export function registerSessionRoutes(
|
|||||||
const rows = await sql<Session[]>`
|
const rows = await sql<Session[]>`
|
||||||
UPDATE sessions SET status = 'open', updated_at = clock_timestamp()
|
UPDATE sessions SET status = 'open', updated_at = clock_timestamp()
|
||||||
WHERE id = ${req.params.id} AND status = 'archived'
|
WHERE id = ${req.params.id} AND status = 'archived'
|
||||||
RETURNING id, project_id, name, model, system_prompt, status, created_at, updated_at
|
RETURNING id, project_id, name, model, system_prompt, status, created_at, updated_at, agent_id
|
||||||
`;
|
`;
|
||||||
if (rows.length === 0) {
|
if (rows.length === 0) {
|
||||||
reply.code(404);
|
reply.code(404);
|
||||||
|
|||||||
@@ -153,3 +153,8 @@ BEGIN
|
|||||||
CHECK (status IN ('open', 'archived'));
|
CHECK (status IN ('open', 'archived'));
|
||||||
END IF;
|
END IF;
|
||||||
END $$;
|
END $$;
|
||||||
|
|
||||||
|
-- v1.x-batch9: per-session agent reference. Agent definitions are not stored in
|
||||||
|
-- the DB; they live in builtins (services/agents.ts) and a per-project AGENTS.md.
|
||||||
|
-- agent_id is the slugified agent name. NULL means "use BooCode defaults".
|
||||||
|
ALTER TABLE sessions ADD COLUMN IF NOT EXISTS agent_id TEXT;
|
||||||
|
|||||||
458
apps/server/src/services/agents.ts
Normal file
458
apps/server/src/services/agents.ts
Normal file
@@ -0,0 +1,458 @@
|
|||||||
|
import { promises as fs } from 'node:fs';
|
||||||
|
import { join } from 'node:path';
|
||||||
|
import type { Agent, AgentsResponse } from '../types/api.js';
|
||||||
|
|
||||||
|
// Tools whitelist universe matches services/tools.ts ALL_TOOLS. Keep in sync.
|
||||||
|
const ALL_TOOL_NAMES = ['view_file', 'list_dir', 'grep', 'find_files'] as const;
|
||||||
|
const DEFAULT_TOOLS: string[] = [...ALL_TOOL_NAMES];
|
||||||
|
const DEFAULT_TEMPERATURE = 0.7;
|
||||||
|
|
||||||
|
export function slugify(name: string): string {
|
||||||
|
return name
|
||||||
|
.toLowerCase()
|
||||||
|
.replace(/[^a-z0-9]+/g, '-')
|
||||||
|
.replace(/^-+|-+$/g, '');
|
||||||
|
}
|
||||||
|
|
||||||
|
// Six builtin defaults. model is intentionally null — session.model wins.
|
||||||
|
// Match AGENTS.md format; system prompts are verbatim.
|
||||||
|
const BUILTIN_AGENTS: Agent[] = [
|
||||||
|
{
|
||||||
|
id: 'code-reviewer',
|
||||||
|
name: 'Code Reviewer',
|
||||||
|
description: 'Reviews code for bugs, security issues, and maintainability. Read-only.',
|
||||||
|
temperature: 0.3,
|
||||||
|
tools: [...DEFAULT_TOOLS],
|
||||||
|
model: null,
|
||||||
|
source: 'builtin',
|
||||||
|
system_prompt: `You review code. Find real problems, not style nits.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Read the file(s) in question with view_file. If a diff is provided, read surrounding context too.
|
||||||
|
2. Use grep/find_files to check how changed symbols are used elsewhere.
|
||||||
|
3. Cite every finding as file:line.
|
||||||
|
|
||||||
|
Prioritize in order:
|
||||||
|
1. Bugs and logic errors
|
||||||
|
2. Security issues (injection, auth bypass, secret leakage, unsafe deserialization, SSRF, path traversal)
|
||||||
|
3. Race conditions, error handling, resource leaks
|
||||||
|
4. Performance issues with measurable impact
|
||||||
|
5. Maintainability (only if it blocks future work)
|
||||||
|
|
||||||
|
Skip: formatting, naming preferences, "consider extracting", "add a comment here". The user has a linter.
|
||||||
|
|
||||||
|
Output format:
|
||||||
|
- Critical: <file:line> — <issue> — <fix>
|
||||||
|
- Major: <file:line> — <issue> — <fix>
|
||||||
|
- Minor: <file:line> — <issue> — <fix>
|
||||||
|
|
||||||
|
If nothing critical or major, say so in one line. Do not pad.`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: 'debugger',
|
||||||
|
name: 'Debugger',
|
||||||
|
description: 'Diagnoses bugs from error messages, logs, or described symptoms.',
|
||||||
|
temperature: 0.2,
|
||||||
|
tools: [...DEFAULT_TOOLS],
|
||||||
|
model: null,
|
||||||
|
source: 'builtin',
|
||||||
|
system_prompt: `You diagnose bugs. Form a hypothesis, prove it with evidence from the code.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Restate the symptom in one line. Confirm you understand it.
|
||||||
|
2. Read the error/stacktrace. Identify the exact frame where things go wrong.
|
||||||
|
3. view_file on that frame. Read 50 lines around it.
|
||||||
|
4. grep for callers, related state, recent changes that could explain it.
|
||||||
|
5. State the root cause with file:line evidence.
|
||||||
|
6. Propose the minimal fix. Note any side effects.
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Never guess. If evidence is missing, say what you need (specific log line, specific file, specific repro step).
|
||||||
|
- Distinguish symptom from cause. A null check fixes the symptom; missing init causes it.
|
||||||
|
- Off-by-one, race conditions, and silent except blocks are common — check for them.
|
||||||
|
- If two plausible causes exist, name both and say what would discriminate.
|
||||||
|
|
||||||
|
Output:
|
||||||
|
- Symptom: <one line>
|
||||||
|
- Root cause: <file:line> — <explanation>
|
||||||
|
- Fix: <minimal diff or description>
|
||||||
|
- Risk: <what could break>`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: 'refactorer',
|
||||||
|
name: 'Refactorer',
|
||||||
|
description: 'Proposes refactors for clarity, deduplication, or decoupling. Read-only — outputs plans, not edits.',
|
||||||
|
temperature: 0.3,
|
||||||
|
tools: [...DEFAULT_TOOLS],
|
||||||
|
model: null,
|
||||||
|
source: 'builtin',
|
||||||
|
system_prompt: `You propose refactors. You do not apply them. The user applies via OpenCode or Claude Code.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Read the target file(s).
|
||||||
|
2. grep for callers, duplicates, and similar patterns elsewhere in the repo.
|
||||||
|
3. Identify the smallest refactor that delivers the goal.
|
||||||
|
|
||||||
|
Prioritize:
|
||||||
|
1. Deduplication where 3+ sites have near-identical logic
|
||||||
|
2. Extracting a function/module when one is doing two unrelated jobs
|
||||||
|
3. Decoupling when a change in A forces a change in B unnecessarily
|
||||||
|
4. Renaming when a name actively misleads
|
||||||
|
|
||||||
|
Reject:
|
||||||
|
- Refactors that touch 10+ files for marginal gain
|
||||||
|
- "Modernization" with no concrete benefit
|
||||||
|
- Abstraction for future flexibility that may never come
|
||||||
|
- Style-only changes
|
||||||
|
|
||||||
|
Output:
|
||||||
|
- Goal: <one line>
|
||||||
|
- Scope: <files affected, count of lines roughly>
|
||||||
|
- Plan: numbered steps, each one self-contained
|
||||||
|
- Risk: <what tests must pass, what could regress>
|
||||||
|
- Skip if: <conditions under which this refactor is not worth doing>`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: 'architect',
|
||||||
|
name: 'Architect',
|
||||||
|
description: 'Designs new features, modules, or architectural changes. Outputs a build plan.',
|
||||||
|
temperature: 0.5,
|
||||||
|
tools: [...DEFAULT_TOOLS],
|
||||||
|
model: null,
|
||||||
|
source: 'builtin',
|
||||||
|
system_prompt: `You design. You produce build plans, not code.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Restate the goal in your own words. Confirm constraints (perf, deploy, deps).
|
||||||
|
2. list_dir the relevant areas. Read existing patterns — match them unless there's a reason not to.
|
||||||
|
3. Decide: extend existing code or add new module. Justify.
|
||||||
|
4. Sketch the data flow: inputs → transforms → outputs → side effects.
|
||||||
|
5. Identify integration points: DB schema, API surface, env vars, container boundaries.
|
||||||
|
6. List failure modes and how the design handles them.
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Reuse before inventing. If a service/lib in the repo already does this, say so.
|
||||||
|
- Prefer boring tech. New deps require justification.
|
||||||
|
- Tailscale IPs for internal routing. No 0.0.0.0 binds.
|
||||||
|
- Least privilege: separate read/write paths, explicit auth gates.
|
||||||
|
- State assumptions inline. Do not ask clarifying questions mid-design unless blocked.
|
||||||
|
|
||||||
|
Output:
|
||||||
|
- Goal
|
||||||
|
- Existing code to reuse: <file paths>
|
||||||
|
- New code: <file paths, one-line purpose each>
|
||||||
|
- Data model changes: <SQL or schema diff>
|
||||||
|
- API surface: <endpoints, request/response shapes>
|
||||||
|
- Failure modes: <list>
|
||||||
|
- Build order: numbered, each step 30-90 min`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: 'security-auditor',
|
||||||
|
name: 'Security Auditor',
|
||||||
|
description: 'Audits code for security vulnerabilities. Read-only.',
|
||||||
|
temperature: 0.2,
|
||||||
|
tools: [...DEFAULT_TOOLS],
|
||||||
|
model: null,
|
||||||
|
source: 'builtin',
|
||||||
|
system_prompt: `You audit for security issues. Concrete findings only, no generic warnings.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Identify the trust boundary: where does untrusted input enter? Where does it leave?
|
||||||
|
2. Trace input flow with grep. Mark every transformation.
|
||||||
|
3. Check each finding against a real attack scenario.
|
||||||
|
|
||||||
|
Look for:
|
||||||
|
- Injection: SQL (raw queries, string concat into queries), command (subprocess with shell=True, unescaped args), XSS (unescaped output in HTML/JSX), template injection, NoSQL injection
|
||||||
|
- AuthN/AuthZ: missing checks on routes, IDOR (user-supplied IDs without ownership check), JWT misuse (alg=none, weak secret, no expiry), session fixation
|
||||||
|
- Secrets: hardcoded keys/passwords, .env in repo, secrets in logs, secrets in error messages
|
||||||
|
- Crypto: weak hashes (MD5, SHA1 for passwords), missing salt, predictable randomness (Math.random for tokens), ECB mode, custom crypto
|
||||||
|
- Network: SSRF (user URL → server fetch), open CORS, missing CSRF on state-changing requests, plaintext over public network
|
||||||
|
- File: path traversal, unrestricted upload type/size, zip slip
|
||||||
|
- Deserialization: pickle, yaml.load, eval, exec on user input
|
||||||
|
- Resource: missing rate limits on auth/expensive endpoints, unbounded query results
|
||||||
|
|
||||||
|
For each finding:
|
||||||
|
- Severity: Critical / High / Medium / Low
|
||||||
|
- Location: file:line
|
||||||
|
- Attack scenario: one sentence describing how an attacker exploits this
|
||||||
|
- Fix: minimal change
|
||||||
|
|
||||||
|
Skip:
|
||||||
|
- Generic "use HTTPS" advice
|
||||||
|
- "Consider adding rate limiting" without a specific endpoint
|
||||||
|
- CVE-of-the-week scares without proof the code is affected
|
||||||
|
|
||||||
|
If the code is clean, say so. Do not invent findings.`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
id: 'prompt-builder',
|
||||||
|
name: 'Prompt Builder',
|
||||||
|
description: 'Builds prompts for OpenCode, Claude Code, or BooCode dispatch.',
|
||||||
|
temperature: 0.4,
|
||||||
|
tools: [...DEFAULT_TOOLS],
|
||||||
|
model: null,
|
||||||
|
source: 'builtin',
|
||||||
|
system_prompt: `You write prompts that another coding agent will execute. Your output is the prompt, not the work.
|
||||||
|
|
||||||
|
Process:
|
||||||
|
1. Ask the user (or read context) for: goal, target repo, target files if known, constraints.
|
||||||
|
2. list_dir and view_file the target area. Confirm files exist and are roughly the shape you think.
|
||||||
|
3. Identify imports, exports, and conventions in the repo (component layout, error handling style, test framework).
|
||||||
|
4. Write the prompt.
|
||||||
|
|
||||||
|
Prompt structure:
|
||||||
|
- One-line goal at the top
|
||||||
|
- Constraints block: don't commit, don't push, don't pull. Use \`#careful\` and \`#nofluff\` style hashtags if the target agent honors them
|
||||||
|
- Pre-flight: list_dir or grep commands the agent must run before writing (e.g. "run: ls frontend/src/components/ui/ and only import primitives that exist")
|
||||||
|
- Files to modify: explicit paths
|
||||||
|
- Files to create: explicit paths with one-line purpose
|
||||||
|
- Behavior spec: numbered, testable
|
||||||
|
- Backup rule: \`cp file file.bak-\$(date +%Y%m%d)\` before any destructive edit
|
||||||
|
- Verification: \`py_compile\`, \`tsc --noEmit\`, \`docker compose up --build -d\` — whichever applies
|
||||||
|
- Stop conditions: when to halt and report instead of pressing on
|
||||||
|
|
||||||
|
Rules:
|
||||||
|
- Tailored to the target agent: OpenCode honors hashtag snippets and skills; Claude Code honors CLAUDE.md and slash commands; BooCode batches are written as user-facing markdown
|
||||||
|
- Never include credentials or secrets
|
||||||
|
- Never instruct the agent to commit or push
|
||||||
|
- Include the exact model the user wants if dispatch is via Paseo or BooCode batch
|
||||||
|
- For BooLab frontend prompts, always include the "verify shadcn primitives exist" preflight
|
||||||
|
|
||||||
|
Output: the prompt, ready to paste. Nothing else.`,
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
|
// ---- AGENTS.md parser ------------------------------------------------------
|
||||||
|
|
||||||
|
interface ParsedFrontmatter {
|
||||||
|
temperature?: number;
|
||||||
|
tools?: string[];
|
||||||
|
description?: string;
|
||||||
|
model?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
function stripQuotes(s: string): string {
|
||||||
|
if (
|
||||||
|
s.length >= 2 &&
|
||||||
|
(s[0] === '"' || s[0] === "'") &&
|
||||||
|
s[0] === s[s.length - 1]
|
||||||
|
) {
|
||||||
|
return s.slice(1, -1);
|
||||||
|
}
|
||||||
|
return s;
|
||||||
|
}
|
||||||
|
|
||||||
|
function parseFrontmatter(yaml: string): { data: ParsedFrontmatter; errors: string[] } {
|
||||||
|
const data: ParsedFrontmatter = {};
|
||||||
|
const errors: string[] = [];
|
||||||
|
const lines = yaml.split('\n');
|
||||||
|
let arrayKey: 'tools' | null = null;
|
||||||
|
|
||||||
|
for (const rawLine of lines) {
|
||||||
|
const line = rawLine.trim();
|
||||||
|
if (line.length === 0) continue;
|
||||||
|
|
||||||
|
// Block-list continuation: "- value" under a key that was set to empty
|
||||||
|
if (arrayKey && line.startsWith('- ')) {
|
||||||
|
data[arrayKey]!.push(line.slice(2).trim());
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
arrayKey = null;
|
||||||
|
|
||||||
|
const colonIdx = line.indexOf(':');
|
||||||
|
if (colonIdx < 0) continue;
|
||||||
|
const key = line.slice(0, colonIdx).trim();
|
||||||
|
const valueRaw = line.slice(colonIdx + 1).trim();
|
||||||
|
|
||||||
|
if (key === 'temperature') {
|
||||||
|
const n = Number(valueRaw);
|
||||||
|
if (Number.isFinite(n)) data.temperature = n;
|
||||||
|
else errors.push(`temperature must be a number (got "${valueRaw}")`);
|
||||||
|
} else if (key === 'tools') {
|
||||||
|
if (valueRaw === '') {
|
||||||
|
data.tools = [];
|
||||||
|
arrayKey = 'tools';
|
||||||
|
} else if (valueRaw.startsWith('[') && valueRaw.endsWith(']')) {
|
||||||
|
const inner = valueRaw.slice(1, -1);
|
||||||
|
data.tools = inner
|
||||||
|
.split(',')
|
||||||
|
.map((s) => stripQuotes(s.trim()))
|
||||||
|
.filter((s) => s.length > 0);
|
||||||
|
} else {
|
||||||
|
// Loose form: "tools: a, b, c"
|
||||||
|
data.tools = valueRaw
|
||||||
|
.split(',')
|
||||||
|
.map((s) => stripQuotes(s.trim()))
|
||||||
|
.filter((s) => s.length > 0);
|
||||||
|
}
|
||||||
|
} else if (key === 'description') {
|
||||||
|
data.description = stripQuotes(valueRaw);
|
||||||
|
} else if (key === 'model') {
|
||||||
|
data.model = stripQuotes(valueRaw);
|
||||||
|
}
|
||||||
|
// Unknown keys silently ignored — forward-compat.
|
||||||
|
}
|
||||||
|
|
||||||
|
return { data, errors };
|
||||||
|
}
|
||||||
|
|
||||||
|
interface ParseResult {
|
||||||
|
agents: Agent[];
|
||||||
|
error: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function parseAgentsMd(content: string): ParseResult {
|
||||||
|
const errors: string[] = [];
|
||||||
|
const agents: Agent[] = [];
|
||||||
|
|
||||||
|
// Split into per-agent sections by lines that exactly match "## <name>".
|
||||||
|
// Lines starting with "### " (level-3 headings) are not section boundaries.
|
||||||
|
const sections: { name: string; body: string }[] = [];
|
||||||
|
let currentName: string | null = null;
|
||||||
|
let currentLines: string[] = [];
|
||||||
|
|
||||||
|
for (const line of content.split('\n')) {
|
||||||
|
const h2 = /^##\s+(.+?)\s*$/.exec(line);
|
||||||
|
const h3 = line.startsWith('### ');
|
||||||
|
if (h2 && !h3) {
|
||||||
|
if (currentName !== null) {
|
||||||
|
sections.push({ name: currentName, body: currentLines.join('\n') });
|
||||||
|
}
|
||||||
|
currentName = h2[1]!.trim();
|
||||||
|
currentLines = [];
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (currentName !== null) {
|
||||||
|
currentLines.push(line);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if (currentName !== null) {
|
||||||
|
sections.push({ name: currentName, body: currentLines.join('\n') });
|
||||||
|
}
|
||||||
|
|
||||||
|
for (const section of sections) {
|
||||||
|
const lines = section.body.split('\n');
|
||||||
|
// Opening "---" fence must be the first non-empty line (blank lines allowed).
|
||||||
|
let openIdx = -1;
|
||||||
|
for (let i = 0; i < lines.length; i++) {
|
||||||
|
const t = lines[i]!.trim();
|
||||||
|
if (t === '') continue;
|
||||||
|
if (t === '---') {
|
||||||
|
openIdx = i;
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
if (openIdx < 0) {
|
||||||
|
errors.push(`agent "${section.name}": missing opening --- fence after heading`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
let closeIdx = -1;
|
||||||
|
for (let i = openIdx + 1; i < lines.length; i++) {
|
||||||
|
if (lines[i]!.trim() === '---') {
|
||||||
|
closeIdx = i;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if (closeIdx < 0) {
|
||||||
|
errors.push(`agent "${section.name}": missing closing --- fence`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
const yamlText = lines.slice(openIdx + 1, closeIdx).join('\n');
|
||||||
|
const systemPrompt = lines.slice(closeIdx + 1).join('\n').trim();
|
||||||
|
|
||||||
|
const { data: fm, errors: fmErrors } = parseFrontmatter(yamlText);
|
||||||
|
if (fmErrors.length > 0) {
|
||||||
|
errors.push(`agent "${section.name}": ${fmErrors.join('; ')}`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const filteredTools = Array.isArray(fm.tools)
|
||||||
|
? fm.tools.filter((t): t is string =>
|
||||||
|
(ALL_TOOL_NAMES as readonly string[]).includes(t)
|
||||||
|
)
|
||||||
|
: DEFAULT_TOOLS;
|
||||||
|
|
||||||
|
agents.push({
|
||||||
|
id: slugify(section.name),
|
||||||
|
name: section.name,
|
||||||
|
description: fm.description ?? '',
|
||||||
|
system_prompt: systemPrompt,
|
||||||
|
temperature: typeof fm.temperature === 'number' ? fm.temperature : DEFAULT_TEMPERATURE,
|
||||||
|
tools: filteredTools,
|
||||||
|
model: typeof fm.model === 'string' && fm.model.length > 0 ? fm.model : null,
|
||||||
|
source: 'file',
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
return { agents, error: errors.length > 0 ? errors.join('; ') : null };
|
||||||
|
}
|
||||||
|
|
||||||
|
// ---- mtime-keyed cache + public API ----------------------------------------
|
||||||
|
|
||||||
|
interface CacheEntry {
|
||||||
|
mtimeMs: number;
|
||||||
|
result: AgentsResponse;
|
||||||
|
}
|
||||||
|
|
||||||
|
const cache = new Map<string, CacheEntry>();
|
||||||
|
|
||||||
|
// Test/admin: force re-parse on next call for a project (or all projects).
|
||||||
|
export function invalidateAgentsCache(projectPath?: string): void {
|
||||||
|
if (projectPath === undefined) {
|
||||||
|
cache.clear();
|
||||||
|
} else {
|
||||||
|
cache.delete(projectPath);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
export async function getAgentsForProject(projectPath: string): Promise<AgentsResponse> {
|
||||||
|
const agentsPath = join(projectPath, 'AGENTS.md');
|
||||||
|
let mtimeMs: number;
|
||||||
|
try {
|
||||||
|
const s = await fs.stat(agentsPath);
|
||||||
|
mtimeMs = s.mtimeMs;
|
||||||
|
} catch {
|
||||||
|
// No AGENTS.md → builtins, no parse error
|
||||||
|
cache.delete(projectPath);
|
||||||
|
return { agents: BUILTIN_AGENTS, parse_error: null };
|
||||||
|
}
|
||||||
|
|
||||||
|
const cached = cache.get(projectPath);
|
||||||
|
if (cached && cached.mtimeMs === mtimeMs) {
|
||||||
|
return cached.result;
|
||||||
|
}
|
||||||
|
|
||||||
|
let content: string;
|
||||||
|
try {
|
||||||
|
content = await fs.readFile(agentsPath, 'utf8');
|
||||||
|
} catch {
|
||||||
|
cache.delete(projectPath);
|
||||||
|
return { agents: BUILTIN_AGENTS, parse_error: null };
|
||||||
|
}
|
||||||
|
|
||||||
|
const parsed = parseAgentsMd(content);
|
||||||
|
let result: AgentsResponse;
|
||||||
|
if (parsed.error) {
|
||||||
|
// Parse error: surface in API, fall back to builtins
|
||||||
|
result = { agents: BUILTIN_AGENTS, parse_error: parsed.error };
|
||||||
|
} else if (parsed.agents.length === 0) {
|
||||||
|
// Empty / no headings → builtins
|
||||||
|
result = { agents: BUILTIN_AGENTS, parse_error: null };
|
||||||
|
} else {
|
||||||
|
// At least one valid agent → file-defined agents win, builtins hidden
|
||||||
|
result = { agents: parsed.agents, parse_error: null };
|
||||||
|
}
|
||||||
|
|
||||||
|
cache.set(projectPath, { mtimeMs, result });
|
||||||
|
return result;
|
||||||
|
}
|
||||||
|
|
||||||
|
export async function getAgentById(
|
||||||
|
projectPath: string,
|
||||||
|
agentId: string
|
||||||
|
): Promise<Agent | null> {
|
||||||
|
const { agents } = await getAgentsForProject(projectPath);
|
||||||
|
return agents.find((a) => a.id === agentId) ?? null;
|
||||||
|
}
|
||||||
|
|
||||||
|
export { BUILTIN_AGENTS };
|
||||||
92
apps/server/src/services/git_meta.ts
Normal file
92
apps/server/src/services/git_meta.ts
Normal file
@@ -0,0 +1,92 @@
|
|||||||
|
import { execFile } from 'node:child_process';
|
||||||
|
import { promisify } from 'node:util';
|
||||||
|
|
||||||
|
const execFileAsync = promisify(execFile);
|
||||||
|
|
||||||
|
const CACHE_TTL_MS = 30_000;
|
||||||
|
const GIT_TIMEOUT_MS = 2_000;
|
||||||
|
// Cap stdout size so a pathological repo can't blow the buffer. Branch + status
|
||||||
|
// porcelain + diverge counts never approach this on a real repo.
|
||||||
|
const GIT_MAX_BUFFER = 1024 * 1024;
|
||||||
|
|
||||||
|
export interface GitMeta {
|
||||||
|
branch: string | null;
|
||||||
|
is_dirty: boolean;
|
||||||
|
ahead: number;
|
||||||
|
behind: number;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface CacheEntry {
|
||||||
|
at: number;
|
||||||
|
value: GitMeta | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
const cache = new Map<string, CacheEntry>();
|
||||||
|
|
||||||
|
// Runs a single git invocation with a hard 2s timeout. Returns null on any
|
||||||
|
// failure (non-zero exit, timeout, git not installed) so callers can decide
|
||||||
|
// how to degrade. Stderr is intentionally swallowed; we don't surface git's
|
||||||
|
// error text to the model or UI.
|
||||||
|
async function runGit(args: string[], cwd: string): Promise<string | null> {
|
||||||
|
try {
|
||||||
|
const { stdout } = await execFileAsync('git', args, {
|
||||||
|
cwd,
|
||||||
|
timeout: GIT_TIMEOUT_MS,
|
||||||
|
windowsHide: true,
|
||||||
|
maxBuffer: GIT_MAX_BUFFER,
|
||||||
|
});
|
||||||
|
return stdout.toString();
|
||||||
|
} catch {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
export async function getGitMeta(rootPath: string): Promise<GitMeta | null> {
|
||||||
|
const cached = cache.get(rootPath);
|
||||||
|
const now = Date.now();
|
||||||
|
if (cached && now - cached.at < CACHE_TTL_MS) {
|
||||||
|
return cached.value;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Three calls in parallel. rev-parse establishes repo + branch name;
|
||||||
|
// status --porcelain detects dirtiness with no false-positives from formatting;
|
||||||
|
// rev-list --left-right --count compares HEAD to upstream and is allowed to
|
||||||
|
// fail silently (returns null → ahead/behind = 0) when no upstream is set.
|
||||||
|
const [branchOut, statusOut, divergedOut] = await Promise.all([
|
||||||
|
runGit(['rev-parse', '--abbrev-ref', 'HEAD'], rootPath),
|
||||||
|
runGit(['status', '--porcelain'], rootPath),
|
||||||
|
runGit(['rev-list', '--left-right', '--count', 'HEAD...@{upstream}'], rootPath),
|
||||||
|
]);
|
||||||
|
|
||||||
|
// If rev-parse fails, this isn't a git repo (or git isn't installed). Cache
|
||||||
|
// the null result so the next 30s of requests don't re-probe.
|
||||||
|
if (branchOut === null) {
|
||||||
|
cache.set(rootPath, { at: now, value: null });
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
const branch = branchOut.trim() || null;
|
||||||
|
const is_dirty = statusOut !== null && statusOut.trim().length > 0;
|
||||||
|
|
||||||
|
let ahead = 0;
|
||||||
|
let behind = 0;
|
||||||
|
if (divergedOut !== null) {
|
||||||
|
const match = divergedOut.trim().match(/^(\d+)\s+(\d+)/);
|
||||||
|
if (match) {
|
||||||
|
ahead = Number(match[1]);
|
||||||
|
behind = Number(match[2]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const value: GitMeta = { branch, is_dirty, ahead, behind };
|
||||||
|
cache.set(rootPath, { at: now, value });
|
||||||
|
return value;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function invalidateGitMetaCache(rootPath?: string): void {
|
||||||
|
if (rootPath) {
|
||||||
|
cache.delete(rootPath);
|
||||||
|
} else {
|
||||||
|
cache.clear();
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -1,10 +1,11 @@
|
|||||||
import type { FastifyBaseLogger } from 'fastify';
|
import type { FastifyBaseLogger } from 'fastify';
|
||||||
import type { Sql } from '../db.js';
|
import type { Sql } from '../db.js';
|
||||||
import type { Config } from '../config.js';
|
import type { Config } from '../config.js';
|
||||||
import type { Message, Project, Session, ToolCall, UserStreamFrame } from '../types/api.js';
|
import type { Agent, Message, Project, Session, ToolCall, UserStreamFrame } from '../types/api.js';
|
||||||
import { ALL_TOOLS, TOOLS_BY_NAME, toolJsonSchemas } from './tools.js';
|
import { ALL_TOOLS, TOOLS_BY_NAME, toolJsonSchemas, type ToolJsonSchema } from './tools.js';
|
||||||
import { PathScopeError, resolveProjectRoot } from './path_guard.js';
|
import { PathScopeError, resolveProjectRoot } from './path_guard.js';
|
||||||
import { maybeAutoNameChat } from './auto_name.js';
|
import { maybeAutoNameChat } from './auto_name.js';
|
||||||
|
import { getAgentById } from './agents.js';
|
||||||
|
|
||||||
const BASE_SYSTEM_PROMPT = (projectPath: string) =>
|
const BASE_SYSTEM_PROMPT = (projectPath: string) =>
|
||||||
`You are BooCode Chat, a code investigation assistant. The user is working on a project located at ${projectPath}. Use the file-read tools (view_file, list_dir, grep, find_files) to investigate code when needed. Be concise. Cite file paths and line numbers when discussing code. Do not hallucinate file contents — read the file first. Tool results may be truncated; if so, narrow your query rather than guessing.`;
|
`You are BooCode Chat, a code investigation assistant. The user is working on a project located at ${projectPath}. Use the file-read tools (view_file, list_dir, grep, find_files) to investigate code when needed. Be concise. Cite file paths and line numbers when discussing code. Do not hallucinate file contents — read the file first. Tool results may be truncated; if so, narrow your query rather than guessing.`;
|
||||||
@@ -91,16 +92,32 @@ export interface InferenceContext {
|
|||||||
publishUser: (frame: UserStreamFrame) => void;
|
publishUser: (frame: UserStreamFrame) => void;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Resolution order: base prompt < agent.system_prompt < session.system_prompt.
|
||||||
|
// Agent prompts layer on top of the base; session prompt is the most specific
|
||||||
|
// override and stacks last so callers can append per-session instructions.
|
||||||
|
export function buildSystemPrompt(
|
||||||
|
project: Project,
|
||||||
|
session: Session,
|
||||||
|
agent: Agent | null
|
||||||
|
): string {
|
||||||
|
let out = BASE_SYSTEM_PROMPT(project.path);
|
||||||
|
if (agent && agent.system_prompt.trim().length > 0) {
|
||||||
|
out += '\n\n' + agent.system_prompt.trim();
|
||||||
|
}
|
||||||
|
if (session.system_prompt && session.system_prompt.trim().length > 0) {
|
||||||
|
out += '\n\n' + session.system_prompt.trim();
|
||||||
|
}
|
||||||
|
return out;
|
||||||
|
}
|
||||||
|
|
||||||
export function buildMessagesPayload(
|
export function buildMessagesPayload(
|
||||||
session: Session,
|
session: Session,
|
||||||
project: Project,
|
project: Project,
|
||||||
history: Message[]
|
history: Message[],
|
||||||
|
agent: Agent | null = null
|
||||||
): OpenAiMessage[] {
|
): OpenAiMessage[] {
|
||||||
const out: OpenAiMessage[] = [];
|
const out: OpenAiMessage[] = [];
|
||||||
let systemPrompt = BASE_SYSTEM_PROMPT(project.path);
|
const systemPrompt = buildSystemPrompt(project, session, agent);
|
||||||
if (session.system_prompt && session.system_prompt.trim().length > 0) {
|
|
||||||
systemPrompt += '\n\n' + session.system_prompt.trim();
|
|
||||||
}
|
|
||||||
out.push({ role: 'system', content: systemPrompt });
|
out.push({ role: 'system', content: systemPrompt });
|
||||||
|
|
||||||
// Find the latest compact marker — only send messages from that point onwards
|
// Find the latest compact marker — only send messages from that point onwards
|
||||||
@@ -161,7 +178,7 @@ async function loadContext(
|
|||||||
chatId: string
|
chatId: string
|
||||||
): Promise<{ session: Session; project: Project; history: Message[] } | null> {
|
): Promise<{ session: Session; project: Project; history: Message[] } | null> {
|
||||||
const sessionRows = await sql<Session[]>`
|
const sessionRows = await sql<Session[]>`
|
||||||
SELECT id, project_id, name, model, system_prompt, status, created_at, updated_at
|
SELECT id, project_id, name, model, system_prompt, status, created_at, updated_at, agent_id
|
||||||
FROM sessions WHERE id = ${sessionId}
|
FROM sessions WHERE id = ${sessionId}
|
||||||
`;
|
`;
|
||||||
if (sessionRows.length === 0) return null;
|
if (sessionRows.length === 0) return null;
|
||||||
@@ -217,11 +234,18 @@ interface StreamResult {
|
|||||||
nCtx: number | null;
|
nCtx: number | null;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
interface StreamOptions {
|
||||||
|
// null = omit tools entirely (compact phase); [] = caller stripped all tools
|
||||||
|
// (rare; we still omit from the request body to avoid OpenAI 400).
|
||||||
|
tools: ToolJsonSchema[] | null;
|
||||||
|
temperature?: number;
|
||||||
|
}
|
||||||
|
|
||||||
async function streamCompletion(
|
async function streamCompletion(
|
||||||
ctx: InferenceContext,
|
ctx: InferenceContext,
|
||||||
model: string,
|
model: string,
|
||||||
messages: OpenAiMessage[],
|
messages: OpenAiMessage[],
|
||||||
includeTools: boolean,
|
opts: StreamOptions,
|
||||||
onDelta: (content: string) => void,
|
onDelta: (content: string) => void,
|
||||||
signal?: AbortSignal
|
signal?: AbortSignal
|
||||||
): Promise<StreamResult> {
|
): Promise<StreamResult> {
|
||||||
@@ -231,10 +255,13 @@ async function streamCompletion(
|
|||||||
stream: true,
|
stream: true,
|
||||||
stream_options: { include_usage: true },
|
stream_options: { include_usage: true },
|
||||||
};
|
};
|
||||||
if (includeTools) {
|
if (opts.tools && opts.tools.length > 0) {
|
||||||
body['tools'] = toolJsonSchemas();
|
body['tools'] = opts.tools;
|
||||||
body['tool_choice'] = 'auto';
|
body['tool_choice'] = 'auto';
|
||||||
}
|
}
|
||||||
|
if (typeof opts.temperature === 'number') {
|
||||||
|
body['temperature'] = opts.temperature;
|
||||||
|
}
|
||||||
|
|
||||||
const res = await fetch(`${ctx.config.LLAMA_SWAP_URL}/v1/chat/completions`, {
|
const res = await fetch(`${ctx.config.LLAMA_SWAP_URL}/v1/chat/completions`, {
|
||||||
method: 'POST',
|
method: 'POST',
|
||||||
@@ -366,7 +393,8 @@ async function executeStreamPhase(
|
|||||||
args: TurnArgs,
|
args: TurnArgs,
|
||||||
session: Session,
|
session: Session,
|
||||||
messages: OpenAiMessage[],
|
messages: OpenAiMessage[],
|
||||||
state: StreamPhaseState
|
state: StreamPhaseState,
|
||||||
|
agent: Agent | null
|
||||||
): Promise<StreamResult> {
|
): Promise<StreamResult> {
|
||||||
const { sessionId, chatId, assistantMessageId, signal } = args;
|
const { sessionId, chatId, assistantMessageId, signal } = args;
|
||||||
|
|
||||||
@@ -407,12 +435,20 @@ async function executeStreamPhase(
|
|||||||
}, DB_FLUSH_INTERVAL_MS);
|
}, DB_FLUSH_INTERVAL_MS);
|
||||||
};
|
};
|
||||||
|
|
||||||
|
// Tool whitelist: if an agent is set, filter the global tool list to only the
|
||||||
|
// tool names it allows. Unknown names in agent.tools are dropped silently
|
||||||
|
// (handled here by intersection). When no agent: send all tools.
|
||||||
|
const effectiveTools: ToolJsonSchema[] = agent
|
||||||
|
? toolJsonSchemas().filter((t) => agent.tools.includes(t.function.name))
|
||||||
|
: toolJsonSchemas();
|
||||||
|
const effectiveTemperature = agent?.temperature;
|
||||||
|
|
||||||
try {
|
try {
|
||||||
return await streamCompletion(
|
return await streamCompletion(
|
||||||
ctx,
|
ctx,
|
||||||
session.model,
|
session.model,
|
||||||
messages,
|
messages,
|
||||||
true,
|
{ tools: effectiveTools, temperature: effectiveTemperature },
|
||||||
(delta) => {
|
(delta) => {
|
||||||
state.accumulated += delta;
|
state.accumulated += delta;
|
||||||
ctx.publish(sessionId, {
|
ctx.publish(sessionId, {
|
||||||
@@ -457,6 +493,9 @@ async function handleAbortOrError(
|
|||||||
RETURNING project_id, name, updated_at
|
RETURNING project_id, name, updated_at
|
||||||
`;
|
`;
|
||||||
ctx.publishUser({ type: 'session_updated', session_id: sessionId, project_id: failSessRow!.project_id, name: failSessRow!.name, updated_at: failSessRow!.updated_at });
|
ctx.publishUser({ type: 'session_updated', session_id: sessionId, project_id: failSessRow!.project_id, name: failSessRow!.name, updated_at: failSessRow!.updated_at });
|
||||||
|
// v1.8 mobile-tabs: cancellation is a user-initiated stop, treat as idle;
|
||||||
|
// genuine errors flip the dot red.
|
||||||
|
ctx.publishUser({ type: 'chat_status', chat_id: chatId, status: isAbort ? 'idle' : 'error', at: new Date().toISOString() });
|
||||||
if (isAbort) {
|
if (isAbort) {
|
||||||
ctx.publish(sessionId, {
|
ctx.publish(sessionId, {
|
||||||
type: 'message_complete',
|
type: 'message_complete',
|
||||||
@@ -602,6 +641,7 @@ async function finalizeCompletion(
|
|||||||
RETURNING project_id, name, updated_at
|
RETURNING project_id, name, updated_at
|
||||||
`;
|
`;
|
||||||
ctx.publishUser({ type: 'session_updated', session_id: sessionId, project_id: completeSessRow!.project_id, name: completeSessRow!.name, updated_at: completeSessRow!.updated_at });
|
ctx.publishUser({ type: 'session_updated', session_id: sessionId, project_id: completeSessRow!.project_id, name: completeSessRow!.name, updated_at: completeSessRow!.updated_at });
|
||||||
|
ctx.publishUser({ type: 'chat_status', chat_id: chatId, status: 'idle', at: new Date().toISOString() });
|
||||||
ctx.publish(sessionId, {
|
ctx.publish(sessionId, {
|
||||||
type: 'message_complete',
|
type: 'message_complete',
|
||||||
message_id: assistantMessageId,
|
message_id: assistantMessageId,
|
||||||
@@ -647,6 +687,7 @@ async function runAssistantTurn(
|
|||||||
chat_id: chatId,
|
chat_id: chatId,
|
||||||
error: 'tool loop depth exceeded',
|
error: 'tool loop depth exceeded',
|
||||||
});
|
});
|
||||||
|
ctx.publishUser({ type: 'chat_status', chat_id: chatId, status: 'error', at: new Date().toISOString() });
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -657,12 +698,18 @@ async function runAssistantTurn(
|
|||||||
}
|
}
|
||||||
const { session, project, history } = loaded;
|
const { session, project, history } = loaded;
|
||||||
const projectRoot = await resolveProjectRoot(project.path);
|
const projectRoot = await resolveProjectRoot(project.path);
|
||||||
const messages = buildMessagesPayload(session, project, history);
|
// Agent resolution is per-turn so PATCH agent_id mid-conversation takes
|
||||||
|
// effect on the next message. Unknown agent_id returns null silently —
|
||||||
|
// session falls back to base prompt + all tools + default temperature.
|
||||||
|
const agent = session.agent_id
|
||||||
|
? await getAgentById(project.path, session.agent_id)
|
||||||
|
: null;
|
||||||
|
const messages = buildMessagesPayload(session, project, history, agent);
|
||||||
|
|
||||||
const state: StreamPhaseState = { accumulated: '', startedAt: null };
|
const state: StreamPhaseState = { accumulated: '', startedAt: null };
|
||||||
let result: StreamResult;
|
let result: StreamResult;
|
||||||
try {
|
try {
|
||||||
result = await executeStreamPhase(ctx, args, session, messages, state);
|
result = await executeStreamPhase(ctx, args, session, messages, state, agent);
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
await handleAbortOrError(ctx, args, state.accumulated, err);
|
await handleAbortOrError(ctx, args, state.accumulated, err);
|
||||||
return;
|
return;
|
||||||
@@ -720,7 +767,7 @@ async function runCompact(
|
|||||||
ctx,
|
ctx,
|
||||||
session.model,
|
session.model,
|
||||||
messagesForSummary,
|
messagesForSummary,
|
||||||
false,
|
{ tools: null },
|
||||||
(delta) => {
|
(delta) => {
|
||||||
content += delta;
|
content += delta;
|
||||||
ctx.publish(sessionId, {
|
ctx.publish(sessionId, {
|
||||||
@@ -778,6 +825,9 @@ export function createInferenceRunner(
|
|||||||
...ctx,
|
...ctx,
|
||||||
publishUser: (frame) => publishUserFn(user, frame),
|
publishUser: (frame) => publishUserFn(user, frame),
|
||||||
};
|
};
|
||||||
|
// v1.8 mobile-tabs: announce working before the async loop starts so
|
||||||
|
// every device subscribed to the user channel sees the amber dot.
|
||||||
|
callCtx.publishUser({ type: 'chat_status', chat_id: chatId, status: 'working', at: new Date().toISOString() });
|
||||||
const controller = new AbortController();
|
const controller = new AbortController();
|
||||||
let resolveCompleted!: () => void;
|
let resolveCompleted!: () => void;
|
||||||
const completed = new Promise<void>((res) => { resolveCompleted = res; });
|
const completed = new Promise<void>((res) => { resolveCompleted = res; });
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ import { resolve, basename, relative } from 'node:path';
|
|||||||
import { z } from 'zod';
|
import { z } from 'zod';
|
||||||
import { pathGuard, PathScopeError } from './path_guard.js';
|
import { pathGuard, PathScopeError } from './path_guard.js';
|
||||||
import { grep as fileOpsGrep, findFiles as fileOpsFindFiles } from './file_ops.js';
|
import { grep as fileOpsGrep, findFiles as fileOpsFindFiles } from './file_ops.js';
|
||||||
|
import { getGitMeta } from './git_meta.js';
|
||||||
|
|
||||||
const MAX_FILE_BYTES = 5 * 1024 * 1024;
|
const MAX_FILE_BYTES = 5 * 1024 * 1024;
|
||||||
const DEFAULT_VIEW_LINES = 200;
|
const DEFAULT_VIEW_LINES = 200;
|
||||||
@@ -266,11 +267,45 @@ export const findFiles: ToolDef<FindFilesInputT> = {
|
|||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
|
// v1.8 Level 1 branch awareness: gives the model a read-only view of the
|
||||||
|
// project's git state. No path input — operates on the inference-resolved
|
||||||
|
// project root via getGitMeta. Subprocess runs with a 2s timeout (see git_meta).
|
||||||
|
const GitStatusInput = z.object({}).strict();
|
||||||
|
type GitStatusInputT = z.infer<typeof GitStatusInput>;
|
||||||
|
|
||||||
|
export const gitStatus: ToolDef<GitStatusInputT> = {
|
||||||
|
name: 'git_status',
|
||||||
|
description:
|
||||||
|
"Returns the current git branch, whether the working tree is dirty, and ahead/behind counts vs upstream. Read-only. Use when you need to know which branch the user is currently working on.",
|
||||||
|
inputSchema: GitStatusInput,
|
||||||
|
jsonSchema: {
|
||||||
|
type: 'function',
|
||||||
|
function: {
|
||||||
|
name: 'git_status',
|
||||||
|
description:
|
||||||
|
'Returns the current git branch, dirty flag, and ahead/behind counts vs upstream. Read-only.',
|
||||||
|
parameters: {
|
||||||
|
type: 'object',
|
||||||
|
properties: {},
|
||||||
|
additionalProperties: false,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
async execute(_input, projectRoot) {
|
||||||
|
const meta = await getGitMeta(projectRoot);
|
||||||
|
if (meta === null) {
|
||||||
|
return { repo: false, branch: null, is_dirty: false, ahead: 0, behind: 0 };
|
||||||
|
}
|
||||||
|
return { repo: true, ...meta };
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
export const ALL_TOOLS: ReadonlyArray<ToolDef<unknown>> = [
|
export const ALL_TOOLS: ReadonlyArray<ToolDef<unknown>> = [
|
||||||
viewFile as ToolDef<unknown>,
|
viewFile as ToolDef<unknown>,
|
||||||
listDir as ToolDef<unknown>,
|
listDir as ToolDef<unknown>,
|
||||||
grep as ToolDef<unknown>,
|
grep as ToolDef<unknown>,
|
||||||
findFiles as ToolDef<unknown>,
|
findFiles as ToolDef<unknown>,
|
||||||
|
gitStatus as ToolDef<unknown>,
|
||||||
];
|
];
|
||||||
|
|
||||||
export const TOOLS_BY_NAME: Record<string, ToolDef<unknown>> = Object.fromEntries(
|
export const TOOLS_BY_NAME: Record<string, ToolDef<unknown>> = Object.fromEntries(
|
||||||
|
|||||||
@@ -28,6 +28,27 @@ export interface Session {
|
|||||||
status: SessionStatus;
|
status: SessionStatus;
|
||||||
created_at: string;
|
created_at: string;
|
||||||
updated_at: string;
|
updated_at: string;
|
||||||
|
agent_id: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Agent sources: 'builtin' = baked-in default (services/agents.ts),
|
||||||
|
// 'file' = parsed from project's AGENTS.md.
|
||||||
|
export type AgentSource = 'builtin' | 'file';
|
||||||
|
|
||||||
|
export interface Agent {
|
||||||
|
id: string; // slug of name; stable handle stored in sessions.agent_id
|
||||||
|
name: string;
|
||||||
|
description: string;
|
||||||
|
system_prompt: string;
|
||||||
|
temperature: number;
|
||||||
|
tools: string[]; // whitelist of tool names; empty = no tools allowed
|
||||||
|
model: string | null; // null means "session.model wins"
|
||||||
|
source: AgentSource;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface AgentsResponse {
|
||||||
|
agents: Agent[];
|
||||||
|
parse_error: string | null; // present (non-null) when AGENTS.md exists but failed to parse
|
||||||
}
|
}
|
||||||
|
|
||||||
// KEEP IN SYNC: apps/server/src/schema.sql chats_status_chk
|
// KEEP IN SYNC: apps/server/src/schema.sql chats_status_chk
|
||||||
@@ -225,6 +246,14 @@ export interface ProjectUpdatedFrame {
|
|||||||
project_id: string;
|
project_id: string;
|
||||||
name: string;
|
name: string;
|
||||||
}
|
}
|
||||||
|
// v1.8 mobile-tabs: server can't know about client-side panes, so status
|
||||||
|
// is keyed by chat_id. Frontend dot derives pane status from pane.activeChatId.
|
||||||
|
export interface ChatStatusFrame {
|
||||||
|
type: 'chat_status';
|
||||||
|
chat_id: string;
|
||||||
|
status: 'working' | 'idle' | 'error';
|
||||||
|
at: string;
|
||||||
|
}
|
||||||
export type UserStreamFrame =
|
export type UserStreamFrame =
|
||||||
| ProjectCreatedFrame
|
| ProjectCreatedFrame
|
||||||
| ProjectDeletedFrame
|
| ProjectDeletedFrame
|
||||||
@@ -240,4 +269,5 @@ export type UserStreamFrame =
|
|||||||
| ChatDeletedFrame
|
| ChatDeletedFrame
|
||||||
| ProjectArchivedFrame
|
| ProjectArchivedFrame
|
||||||
| ProjectUnarchivedFrame
|
| ProjectUnarchivedFrame
|
||||||
| ProjectUpdatedFrame;
|
| ProjectUpdatedFrame
|
||||||
|
| ChatStatusFrame;
|
||||||
|
|||||||
@@ -8,6 +8,8 @@ import type {
|
|||||||
SidebarResponse,
|
SidebarResponse,
|
||||||
ListDirResult,
|
ListDirResult,
|
||||||
ViewFileResult,
|
ViewFileResult,
|
||||||
|
AgentsResponse,
|
||||||
|
GitMeta,
|
||||||
} from './types';
|
} from './types';
|
||||||
|
|
||||||
export class ApiError extends Error {
|
export class ApiError extends Error {
|
||||||
@@ -86,6 +88,8 @@ export const api = {
|
|||||||
request<ViewFileResult>(`/api/projects/${id}/view_file?path=${encodeURIComponent(path)}`),
|
request<ViewFileResult>(`/api/projects/${id}/view_file?path=${encodeURIComponent(path)}`),
|
||||||
files: (id: string) =>
|
files: (id: string) =>
|
||||||
request<{ files: string[] }>(`/api/projects/${id}/files`),
|
request<{ files: string[] }>(`/api/projects/${id}/files`),
|
||||||
|
git: (id: string) =>
|
||||||
|
request<GitMeta>(`/api/projects/${id}/git`),
|
||||||
},
|
},
|
||||||
|
|
||||||
sessions: {
|
sessions: {
|
||||||
@@ -93,7 +97,7 @@ export const api = {
|
|||||||
request<Session[]>(`/api/projects/${projectId}/sessions${status ? `?status=${status}` : ''}`),
|
request<Session[]>(`/api/projects/${projectId}/sessions${status ? `?status=${status}` : ''}`),
|
||||||
create: (
|
create: (
|
||||||
projectId: string,
|
projectId: string,
|
||||||
body: { name?: string; model?: string; system_prompt?: string }
|
body: { name?: string; model?: string; system_prompt?: string; agent_id?: string | null }
|
||||||
) =>
|
) =>
|
||||||
request<Session>(`/api/projects/${projectId}/sessions`, {
|
request<Session>(`/api/projects/${projectId}/sessions`, {
|
||||||
method: 'POST',
|
method: 'POST',
|
||||||
@@ -102,7 +106,7 @@ export const api = {
|
|||||||
get: (id: string) => request<Session>(`/api/sessions/${id}`),
|
get: (id: string) => request<Session>(`/api/sessions/${id}`),
|
||||||
update: (
|
update: (
|
||||||
id: string,
|
id: string,
|
||||||
body: Partial<Pick<Session, 'name' | 'model' | 'system_prompt'>>
|
body: Partial<Pick<Session, 'name' | 'model' | 'system_prompt' | 'agent_id'>>
|
||||||
) =>
|
) =>
|
||||||
request<Session>(`/api/sessions/${id}`, {
|
request<Session>(`/api/sessions/${id}`, {
|
||||||
method: 'PATCH',
|
method: 'PATCH',
|
||||||
@@ -179,6 +183,11 @@ export const api = {
|
|||||||
|
|
||||||
models: () => request<ModelInfo[]>('/api/models'),
|
models: () => request<ModelInfo[]>('/api/models'),
|
||||||
|
|
||||||
|
agents: {
|
||||||
|
list: (projectId: string) =>
|
||||||
|
request<AgentsResponse>(`/api/projects/${projectId}/agents`),
|
||||||
|
},
|
||||||
|
|
||||||
settings: {
|
settings: {
|
||||||
get: () => request<Record<string, unknown>>('/api/settings'),
|
get: () => request<Record<string, unknown>>('/api/settings'),
|
||||||
patch: (body: Record<string, unknown>) =>
|
patch: (body: Record<string, unknown>) =>
|
||||||
|
|||||||
@@ -27,6 +27,25 @@ export interface Session {
|
|||||||
status: SessionStatus;
|
status: SessionStatus;
|
||||||
created_at: string;
|
created_at: string;
|
||||||
updated_at: string;
|
updated_at: string;
|
||||||
|
agent_id: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
export type AgentSource = 'builtin' | 'file';
|
||||||
|
|
||||||
|
export interface Agent {
|
||||||
|
id: string;
|
||||||
|
name: string;
|
||||||
|
description: string;
|
||||||
|
system_prompt: string;
|
||||||
|
temperature: number;
|
||||||
|
tools: string[];
|
||||||
|
model: string | null;
|
||||||
|
source: AgentSource;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface AgentsResponse {
|
||||||
|
agents: Agent[];
|
||||||
|
parse_error: string | null;
|
||||||
}
|
}
|
||||||
|
|
||||||
export const CHAT_STATUSES = ['open', 'archived'] as const;
|
export const CHAT_STATUSES = ['open', 'archived'] as const;
|
||||||
@@ -156,6 +175,15 @@ export interface PaneUpdateRequest {
|
|||||||
position?: number;
|
position?: number;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// v1.8 mobile-tabs: shape returned by GET /api/projects/:id/git. Mirrors
|
||||||
|
// services/git_meta.ts on the server. branch=null means "not a git repo".
|
||||||
|
export interface GitMeta {
|
||||||
|
branch: string | null;
|
||||||
|
is_dirty: boolean;
|
||||||
|
ahead: number;
|
||||||
|
behind: number;
|
||||||
|
}
|
||||||
|
|
||||||
export type WorkspacePaneKind = 'chat' | 'terminal' | 'agent' | 'empty';
|
export type WorkspacePaneKind = 'chat' | 'terminal' | 'agent' | 'empty';
|
||||||
|
|
||||||
export interface WorkspacePane {
|
export interface WorkspacePane {
|
||||||
|
|||||||
108
apps/web/src/components/AgentPicker.tsx
Normal file
108
apps/web/src/components/AgentPicker.tsx
Normal file
@@ -0,0 +1,108 @@
|
|||||||
|
import { useEffect, useState } from 'react';
|
||||||
|
import { Check, ChevronDown } from 'lucide-react';
|
||||||
|
import { toast } from 'sonner';
|
||||||
|
import { api } from '@/api/client';
|
||||||
|
import type { Agent } from '@/api/types';
|
||||||
|
import {
|
||||||
|
DropdownMenu,
|
||||||
|
DropdownMenuContent,
|
||||||
|
DropdownMenuItem,
|
||||||
|
DropdownMenuSeparator,
|
||||||
|
DropdownMenuTrigger,
|
||||||
|
} from '@/components/ui/dropdown-menu';
|
||||||
|
|
||||||
|
interface Props {
|
||||||
|
projectId: string;
|
||||||
|
value: string | null;
|
||||||
|
onChange: (agentId: string | null) => void | Promise<void>;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function AgentPicker({ projectId, value, onChange }: Props) {
|
||||||
|
const [agents, setAgents] = useState<Agent[] | null>(null);
|
||||||
|
const [error, setError] = useState<string | null>(null);
|
||||||
|
const [open, setOpen] = useState(false);
|
||||||
|
|
||||||
|
// Load on mount (and on projectId change) so the trigger shows the agent
|
||||||
|
// name immediately, not the raw id. AGENTS.md parse errors surface as a
|
||||||
|
// toast once per load.
|
||||||
|
useEffect(() => {
|
||||||
|
let cancelled = false;
|
||||||
|
setAgents(null);
|
||||||
|
setError(null);
|
||||||
|
api.agents
|
||||||
|
.list(projectId)
|
||||||
|
.then((res) => {
|
||||||
|
if (cancelled) return;
|
||||||
|
setAgents(res.agents);
|
||||||
|
if (res.parse_error) {
|
||||||
|
toast.error(`AGENTS.md parse error: ${res.parse_error}`);
|
||||||
|
}
|
||||||
|
})
|
||||||
|
.catch((err) => {
|
||||||
|
if (cancelled) return;
|
||||||
|
setError(err instanceof Error ? err.message : 'failed to load agents');
|
||||||
|
});
|
||||||
|
return () => {
|
||||||
|
cancelled = true;
|
||||||
|
};
|
||||||
|
}, [projectId]);
|
||||||
|
|
||||||
|
const selectedAgent = agents?.find((a) => a.id === value) ?? null;
|
||||||
|
const triggerLabel = value === null
|
||||||
|
? 'No agent'
|
||||||
|
: selectedAgent?.name ?? value;
|
||||||
|
|
||||||
|
return (
|
||||||
|
<DropdownMenu open={open} onOpenChange={setOpen}>
|
||||||
|
<DropdownMenuTrigger asChild>
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
className="text-xs text-muted-foreground hover:text-foreground flex items-center gap-1 px-1.5 py-0.5 rounded hover:bg-muted/60"
|
||||||
|
title={selectedAgent?.description ?? undefined}
|
||||||
|
>
|
||||||
|
<span className="truncate max-w-[160px]">{triggerLabel}</span>
|
||||||
|
<ChevronDown className="size-3 opacity-70" />
|
||||||
|
</button>
|
||||||
|
</DropdownMenuTrigger>
|
||||||
|
<DropdownMenuContent align="start" className="max-h-80 overflow-y-auto w-72">
|
||||||
|
{error && (
|
||||||
|
<div className="px-2 py-1.5 text-xs text-destructive">{error}</div>
|
||||||
|
)}
|
||||||
|
{agents === null && !error && (
|
||||||
|
<div className="px-2 py-1.5 text-xs text-muted-foreground">Loading…</div>
|
||||||
|
)}
|
||||||
|
{agents !== null && (
|
||||||
|
<>
|
||||||
|
<DropdownMenuItem
|
||||||
|
onSelect={() => void onChange(null)}
|
||||||
|
className="text-xs"
|
||||||
|
>
|
||||||
|
<Check className={`size-3 ${value === null ? 'opacity-100' : 'opacity-0'}`} />
|
||||||
|
<span className="font-medium">No agent</span>
|
||||||
|
</DropdownMenuItem>
|
||||||
|
{agents.length > 0 && <DropdownMenuSeparator />}
|
||||||
|
{agents.map((a) => (
|
||||||
|
<DropdownMenuItem
|
||||||
|
key={a.id}
|
||||||
|
onSelect={() => void onChange(a.id)}
|
||||||
|
className="text-xs flex-col items-start gap-0.5"
|
||||||
|
>
|
||||||
|
<div className="flex items-center gap-1.5">
|
||||||
|
<Check
|
||||||
|
className={`size-3 ${a.id === value ? 'opacity-100' : 'opacity-0'}`}
|
||||||
|
/>
|
||||||
|
<span className="font-medium">{a.name}</span>
|
||||||
|
</div>
|
||||||
|
{a.description && (
|
||||||
|
<span className="text-muted-foreground pl-[18px] truncate w-full">
|
||||||
|
{a.description}
|
||||||
|
</span>
|
||||||
|
)}
|
||||||
|
</DropdownMenuItem>
|
||||||
|
))}
|
||||||
|
</>
|
||||||
|
)}
|
||||||
|
</DropdownMenuContent>
|
||||||
|
</DropdownMenu>
|
||||||
|
);
|
||||||
|
}
|
||||||
92
apps/web/src/components/BottomSheet.tsx
Normal file
92
apps/web/src/components/BottomSheet.tsx
Normal file
@@ -0,0 +1,92 @@
|
|||||||
|
import { useEffect, useRef, useState, type ReactNode, type TouchEvent } from 'react';
|
||||||
|
import { cn } from '@/lib/utils';
|
||||||
|
|
||||||
|
interface Props {
|
||||||
|
open: boolean;
|
||||||
|
onClose: () => void;
|
||||||
|
children: ReactNode;
|
||||||
|
title?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Past this drag distance, release dismisses the sheet.
|
||||||
|
const SWIPE_DISMISS_THRESHOLD_PX = 80;
|
||||||
|
|
||||||
|
export function BottomSheet({ open, onClose, children, title }: Props) {
|
||||||
|
const [dragY, setDragY] = useState(0);
|
||||||
|
const startYRef = useRef<number | null>(null);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
if (!open) return;
|
||||||
|
const onKey = (e: KeyboardEvent) => {
|
||||||
|
if (e.key === 'Escape') onClose();
|
||||||
|
};
|
||||||
|
window.addEventListener('keydown', onKey);
|
||||||
|
return () => window.removeEventListener('keydown', onKey);
|
||||||
|
}, [open, onClose]);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
if (!open) {
|
||||||
|
setDragY(0);
|
||||||
|
startYRef.current = null;
|
||||||
|
}
|
||||||
|
}, [open]);
|
||||||
|
|
||||||
|
function onTouchStart(e: TouchEvent<HTMLDivElement>) {
|
||||||
|
const t = e.touches[0];
|
||||||
|
if (!t) return;
|
||||||
|
startYRef.current = t.clientY;
|
||||||
|
}
|
||||||
|
function onTouchMove(e: TouchEvent<HTMLDivElement>) {
|
||||||
|
const t = e.touches[0];
|
||||||
|
if (!t || startYRef.current === null) return;
|
||||||
|
const dy = t.clientY - startYRef.current;
|
||||||
|
// Clamp to downward drags so the sheet doesn't "rubber-band" up.
|
||||||
|
if (dy > 0) setDragY(dy);
|
||||||
|
}
|
||||||
|
function onTouchEnd() {
|
||||||
|
if (dragY > SWIPE_DISMISS_THRESHOLD_PX) {
|
||||||
|
onClose();
|
||||||
|
} else {
|
||||||
|
setDragY(0);
|
||||||
|
}
|
||||||
|
startYRef.current = null;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!open) return null;
|
||||||
|
|
||||||
|
return (
|
||||||
|
<>
|
||||||
|
<div
|
||||||
|
className="fixed inset-0 z-40 bg-black/40"
|
||||||
|
onClick={onClose}
|
||||||
|
aria-hidden="true"
|
||||||
|
/>
|
||||||
|
<div
|
||||||
|
role="dialog"
|
||||||
|
aria-modal="true"
|
||||||
|
className={cn(
|
||||||
|
'fixed inset-x-0 bottom-0 z-50 rounded-t-2xl border-t border-border bg-popover text-popover-foreground shadow-2xl',
|
||||||
|
'transition-transform duration-150 will-change-transform',
|
||||||
|
'max-h-[70vh] flex flex-col',
|
||||||
|
)}
|
||||||
|
style={{
|
||||||
|
transform: `translateY(${dragY}px)`,
|
||||||
|
paddingBottom: 'env(safe-area-inset-bottom)',
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<div
|
||||||
|
onTouchStart={onTouchStart}
|
||||||
|
onTouchMove={onTouchMove}
|
||||||
|
onTouchEnd={onTouchEnd}
|
||||||
|
className="flex flex-col items-center pt-2 pb-1 select-none touch-none"
|
||||||
|
>
|
||||||
|
<div className="w-10 h-1 bg-muted-foreground/40 rounded-full" />
|
||||||
|
{title && (
|
||||||
|
<div className="mt-1 text-sm font-medium text-muted-foreground">{title}</div>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
<div className="flex-1 overflow-y-auto">{children}</div>
|
||||||
|
</div>
|
||||||
|
</>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -15,6 +15,7 @@ import { AttachmentChip } from '@/components/AttachmentChip';
|
|||||||
import { AttachmentPreviewModal } from '@/components/AttachmentPreviewModal';
|
import { AttachmentPreviewModal } from '@/components/AttachmentPreviewModal';
|
||||||
import { FileMentionPopover } from '@/components/FileMentionPopover';
|
import { FileMentionPopover } from '@/components/FileMentionPopover';
|
||||||
import { DropOverlay } from '@/components/DropOverlay';
|
import { DropOverlay } from '@/components/DropOverlay';
|
||||||
|
import { AgentPicker } from '@/components/AgentPicker';
|
||||||
import { api } from '@/api/client';
|
import { api } from '@/api/client';
|
||||||
import { sessionEvents } from '@/hooks/sessionEvents';
|
import { sessionEvents } from '@/hooks/sessionEvents';
|
||||||
import { useViewport } from '@/hooks/useViewport';
|
import { useViewport } from '@/hooks/useViewport';
|
||||||
@@ -24,11 +25,15 @@ const MAX_ATTACHMENTS = 10;
|
|||||||
interface Props {
|
interface Props {
|
||||||
disabled?: boolean;
|
disabled?: boolean;
|
||||||
projectId: string;
|
projectId: string;
|
||||||
|
// Batch 9: optional so callers that pre-date the agent picker still compile.
|
||||||
|
// When omitted, the toolbar row is hidden entirely.
|
||||||
|
agentId?: string | null;
|
||||||
|
onAgentChange?: (agentId: string | null) => void | Promise<void>;
|
||||||
onSend: (content: string) => void | Promise<void>;
|
onSend: (content: string) => void | Promise<void>;
|
||||||
onForceSend?: (content: string) => void | Promise<void>;
|
onForceSend?: (content: string) => void | Promise<void>;
|
||||||
}
|
}
|
||||||
|
|
||||||
export function ChatInput({ disabled, projectId, onSend, onForceSend }: Props) {
|
export function ChatInput({ disabled, projectId, agentId, onAgentChange, onSend, onForceSend }: Props) {
|
||||||
const { isMobile } = useViewport();
|
const { isMobile } = useViewport();
|
||||||
const [value, setValue] = useState('');
|
const [value, setValue] = useState('');
|
||||||
const [busy, setBusy] = useState(false);
|
const [busy, setBusy] = useState(false);
|
||||||
@@ -420,6 +425,18 @@ export function ChatInput({ disabled, projectId, onSend, onForceSend }: Props) {
|
|||||||
))}
|
))}
|
||||||
</div>
|
</div>
|
||||||
)}
|
)}
|
||||||
|
{/* Batch 9 toolbar — agent picker. Sits above the input row so it
|
||||||
|
doesn't compete with the send button for vertical alignment.
|
||||||
|
When Batch 7 lands, ModelPicker and the + button join this row. */}
|
||||||
|
{onAgentChange && (
|
||||||
|
<div className="px-4 pt-2 flex items-center gap-1.5">
|
||||||
|
<AgentPicker
|
||||||
|
projectId={projectId}
|
||||||
|
value={agentId ?? null}
|
||||||
|
onChange={onAgentChange}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
<div className="px-4 py-3 flex items-end gap-2">
|
<div className="px-4 py-3 flex items-end gap-2">
|
||||||
<Textarea
|
<Textarea
|
||||||
ref={textareaRef}
|
ref={textareaRef}
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
import { useState } from 'react';
|
import { useState } from 'react';
|
||||||
import { History, MessageSquare, Plus, X } from 'lucide-react';
|
import { History, MessageSquare, Plus, X } from 'lucide-react';
|
||||||
import type { Chat, WorkspacePane } from '@/api/types';
|
import type { Chat, WorkspacePane } from '@/api/types';
|
||||||
|
import { StatusDot } from '@/components/StatusDot';
|
||||||
import {
|
import {
|
||||||
ContextMenu,
|
ContextMenu,
|
||||||
ContextMenuContent,
|
ContextMenuContent,
|
||||||
@@ -66,7 +67,7 @@ export function ChatTabBar({
|
|||||||
}
|
}
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="flex items-center border-b border-border bg-muted/20 h-8 shrink-0 overflow-x-auto">
|
<div className="flex items-center border-b border-border bg-muted/20 h-8 shrink-0 overflow-x-auto max-md:hidden">
|
||||||
{tabs.map((chat, tabIdx) => {
|
{tabs.map((chat, tabIdx) => {
|
||||||
const isActive = tabIdx === pane.activeChatIdx;
|
const isActive = tabIdx === pane.activeChatIdx;
|
||||||
const isLast = tabIdx === tabs.length - 1;
|
const isLast = tabIdx === tabs.length - 1;
|
||||||
@@ -91,6 +92,7 @@ export function ChatTabBar({
|
|||||||
)}
|
)}
|
||||||
>
|
>
|
||||||
<MessageSquare size={12} className="shrink-0" />
|
<MessageSquare size={12} className="shrink-0" />
|
||||||
|
<StatusDot chatId={chat.id} />
|
||||||
{renamingId === chat.id ? (
|
{renamingId === chat.id ? (
|
||||||
<input
|
<input
|
||||||
autoFocus
|
autoFocus
|
||||||
|
|||||||
207
apps/web/src/components/MobileTabSwitcher.tsx
Normal file
207
apps/web/src/components/MobileTabSwitcher.tsx
Normal file
@@ -0,0 +1,207 @@
|
|||||||
|
import { useState } from 'react';
|
||||||
|
import {
|
||||||
|
Bot,
|
||||||
|
ChevronDown,
|
||||||
|
Edit2,
|
||||||
|
MessageSquare,
|
||||||
|
MoreHorizontal,
|
||||||
|
Terminal,
|
||||||
|
X,
|
||||||
|
} from 'lucide-react';
|
||||||
|
import { toast } from 'sonner';
|
||||||
|
import type { Chat, WorkspacePane } from '@/api/types';
|
||||||
|
import { BottomSheet } from '@/components/BottomSheet';
|
||||||
|
import { StatusDot } from '@/components/StatusDot';
|
||||||
|
import {
|
||||||
|
DropdownMenu,
|
||||||
|
DropdownMenuContent,
|
||||||
|
DropdownMenuItem,
|
||||||
|
DropdownMenuTrigger,
|
||||||
|
} from '@/components/ui/dropdown-menu';
|
||||||
|
import { useLongPress } from '@/hooks/useLongPress';
|
||||||
|
import { cn } from '@/lib/utils';
|
||||||
|
|
||||||
|
interface Props {
|
||||||
|
panes: WorkspacePane[];
|
||||||
|
activePaneIdx: number;
|
||||||
|
chats: Chat[];
|
||||||
|
onSwitchPane: (idx: number) => void;
|
||||||
|
onRemovePane: (idx: number) => void;
|
||||||
|
onRenameChat: (chatId: string, name: string) => Promise<void>;
|
||||||
|
}
|
||||||
|
|
||||||
|
function paneIcon(kind: WorkspacePane['kind']) {
|
||||||
|
if (kind === 'terminal') return <Terminal size={14} />;
|
||||||
|
if (kind === 'agent') return <Bot size={14} />;
|
||||||
|
return <MessageSquare size={14} />;
|
||||||
|
}
|
||||||
|
|
||||||
|
function paneActiveChatId(pane: WorkspacePane | undefined): string | null {
|
||||||
|
if (!pane) return null;
|
||||||
|
if (pane.chatId) return pane.chatId;
|
||||||
|
const idx = pane.activeChatIdx;
|
||||||
|
if (idx < 0 || idx >= pane.chatIds.length) return null;
|
||||||
|
return pane.chatIds[idx] ?? null;
|
||||||
|
}
|
||||||
|
|
||||||
|
function paneLabel(pane: WorkspacePane, chats: Chat[]): string {
|
||||||
|
const cid = paneActiveChatId(pane);
|
||||||
|
if (cid) {
|
||||||
|
const c = chats.find((x) => x.id === cid);
|
||||||
|
if (c) return c.name ?? 'New chat';
|
||||||
|
}
|
||||||
|
if (pane.kind === 'chat') return 'Chat';
|
||||||
|
if (pane.kind === 'terminal') return 'Terminal';
|
||||||
|
if (pane.kind === 'agent') return 'Agent';
|
||||||
|
return 'Empty';
|
||||||
|
}
|
||||||
|
|
||||||
|
export function MobileTabSwitcher({
|
||||||
|
panes,
|
||||||
|
activePaneIdx,
|
||||||
|
chats,
|
||||||
|
onSwitchPane,
|
||||||
|
onRemovePane,
|
||||||
|
onRenameChat,
|
||||||
|
}: Props) {
|
||||||
|
const [open, setOpen] = useState(false);
|
||||||
|
const [renamingChatId, setRenamingChatId] = useState<string | null>(null);
|
||||||
|
const [renameValue, setRenameValue] = useState('');
|
||||||
|
|
||||||
|
const active = panes[activePaneIdx];
|
||||||
|
const activeLabel = active ? paneLabel(active, chats) : 'Empty';
|
||||||
|
const activeChatId = paneActiveChatId(active);
|
||||||
|
|
||||||
|
// Long-press mirrors ChatTabBar: synthesize a contextmenu event on the row
|
||||||
|
// so the trailing kebab's Radix DropdownMenu opens at the touch point.
|
||||||
|
const longPress = useLongPress(({ clientX, clientY, target }) => {
|
||||||
|
if (!target || !(target instanceof Element)) return;
|
||||||
|
const row = target.closest('[data-pane-id]') as HTMLElement | null;
|
||||||
|
if (!row) return;
|
||||||
|
const trigger = row.querySelector('[data-pane-kebab]') as HTMLElement | null;
|
||||||
|
if (trigger) {
|
||||||
|
trigger.click();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
row.dispatchEvent(
|
||||||
|
new MouseEvent('contextmenu', { bubbles: true, cancelable: true, clientX, clientY }),
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
function startRename(chatId: string, currentName: string | null) {
|
||||||
|
setRenamingChatId(chatId);
|
||||||
|
setRenameValue(currentName ?? '');
|
||||||
|
}
|
||||||
|
async function finishRename() {
|
||||||
|
if (renamingChatId && renameValue.trim()) {
|
||||||
|
try {
|
||||||
|
await onRenameChat(renamingChatId, renameValue.trim());
|
||||||
|
} catch (err) {
|
||||||
|
toast.error(err instanceof Error ? err.message : 'rename failed');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
setRenamingChatId(null);
|
||||||
|
}
|
||||||
|
|
||||||
|
function handleSwitchPane(idx: number) {
|
||||||
|
onSwitchPane(idx);
|
||||||
|
setOpen(false);
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
|
<>
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
onClick={() => setOpen(true)}
|
||||||
|
className="flex-1 inline-flex items-center gap-1.5 min-h-[44px] px-3 text-sm rounded-full bg-muted/40 hover:bg-muted/70 text-foreground min-w-0"
|
||||||
|
aria-label="Switch pane"
|
||||||
|
>
|
||||||
|
<span className="shrink-0 text-muted-foreground">{paneIcon(active?.kind ?? 'chat')}</span>
|
||||||
|
<StatusDot chatId={activeChatId} />
|
||||||
|
<span className="truncate flex-1 text-left">{activeLabel}</span>
|
||||||
|
<ChevronDown size={14} className="opacity-60 shrink-0" />
|
||||||
|
</button>
|
||||||
|
|
||||||
|
<BottomSheet open={open} onClose={() => setOpen(false)} title="Panes">
|
||||||
|
<ul className="px-2 py-2 space-y-1">
|
||||||
|
{panes.map((pane, idx) => {
|
||||||
|
const isActive = idx === activePaneIdx;
|
||||||
|
const cid = paneActiveChatId(pane);
|
||||||
|
const chat = cid ? chats.find((c) => c.id === cid) ?? null : null;
|
||||||
|
const label = paneLabel(pane, chats);
|
||||||
|
return (
|
||||||
|
<li
|
||||||
|
key={pane.id}
|
||||||
|
data-pane-id={pane.id}
|
||||||
|
onTouchStart={longPress.onTouchStart}
|
||||||
|
onTouchMove={longPress.onTouchMove}
|
||||||
|
onTouchEnd={longPress.onTouchEnd}
|
||||||
|
onTouchCancel={longPress.onTouchCancel}
|
||||||
|
onClick={() => handleSwitchPane(idx)}
|
||||||
|
style={{ WebkitTouchCallout: 'none' }}
|
||||||
|
className={cn(
|
||||||
|
'flex items-center gap-2 px-3 py-2 rounded min-h-[48px] cursor-default select-none',
|
||||||
|
isActive
|
||||||
|
? 'bg-accent/40 border-l-2 border-primary'
|
||||||
|
: 'hover:bg-muted/50',
|
||||||
|
)}
|
||||||
|
>
|
||||||
|
<span className="shrink-0 text-muted-foreground">{paneIcon(pane.kind)}</span>
|
||||||
|
<StatusDot chatId={cid ?? null} />
|
||||||
|
{renamingChatId === cid && cid ? (
|
||||||
|
<input
|
||||||
|
autoFocus
|
||||||
|
value={renameValue}
|
||||||
|
onChange={(e) => setRenameValue(e.target.value)}
|
||||||
|
onBlur={() => void finishRename()}
|
||||||
|
onKeyDown={(e) => {
|
||||||
|
if (e.key === 'Enter') void finishRename();
|
||||||
|
if (e.key === 'Escape') setRenamingChatId(null);
|
||||||
|
}}
|
||||||
|
onClick={(e) => e.stopPropagation()}
|
||||||
|
className="bg-transparent border-b border-border text-sm outline-none flex-1 min-w-0"
|
||||||
|
/>
|
||||||
|
) : (
|
||||||
|
<span className="truncate flex-1 text-sm">{label}</span>
|
||||||
|
)}
|
||||||
|
{isActive && (
|
||||||
|
<span aria-hidden="true" className="text-primary text-xs shrink-0">
|
||||||
|
✓
|
||||||
|
</span>
|
||||||
|
)}
|
||||||
|
<DropdownMenu>
|
||||||
|
<DropdownMenuTrigger asChild>
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
data-pane-kebab
|
||||||
|
onClick={(e) => e.stopPropagation()}
|
||||||
|
className="inline-flex items-center justify-center p-1 rounded text-muted-foreground hover:bg-muted hover:text-foreground min-h-[44px] min-w-[44px]"
|
||||||
|
aria-label="Pane options"
|
||||||
|
>
|
||||||
|
<MoreHorizontal size={14} />
|
||||||
|
</button>
|
||||||
|
</DropdownMenuTrigger>
|
||||||
|
<DropdownMenuContent align="end">
|
||||||
|
{chat && (
|
||||||
|
<DropdownMenuItem onSelect={() => startRename(chat.id, chat.name)}>
|
||||||
|
<Edit2 size={14} /> Rename chat
|
||||||
|
</DropdownMenuItem>
|
||||||
|
)}
|
||||||
|
<DropdownMenuItem
|
||||||
|
disabled={panes.length <= 1}
|
||||||
|
onSelect={() => onRemovePane(idx)}
|
||||||
|
>
|
||||||
|
<X size={14} /> Close pane
|
||||||
|
</DropdownMenuItem>
|
||||||
|
</DropdownMenuContent>
|
||||||
|
</DropdownMenu>
|
||||||
|
</li>
|
||||||
|
);
|
||||||
|
})}
|
||||||
|
</ul>
|
||||||
|
{/* v1.8: New-pane button moved out of the sheet to the header row 2
|
||||||
|
(see NewPaneMenu). Sheet is for switching only. */}
|
||||||
|
</BottomSheet>
|
||||||
|
</>
|
||||||
|
);
|
||||||
|
}
|
||||||
44
apps/web/src/components/NewPaneMenu.tsx
Normal file
44
apps/web/src/components/NewPaneMenu.tsx
Normal file
@@ -0,0 +1,44 @@
|
|||||||
|
import { Bot, MessageSquare, Plus, Terminal } from 'lucide-react';
|
||||||
|
import {
|
||||||
|
DropdownMenu,
|
||||||
|
DropdownMenuContent,
|
||||||
|
DropdownMenuItem,
|
||||||
|
DropdownMenuTrigger,
|
||||||
|
} from '@/components/ui/dropdown-menu';
|
||||||
|
|
||||||
|
interface Props {
|
||||||
|
onAddPane: (kind: 'chat' | 'terminal' | 'agent') => void;
|
||||||
|
disabled?: boolean;
|
||||||
|
}
|
||||||
|
|
||||||
|
// v1.8 row-2 right cluster: mirrors the desktop Workspace.tsx Split dropdown.
|
||||||
|
// Terminal and Agent items pass through to addSplitPane which already shows
|
||||||
|
// "coming soon" toasts; rendering them here matches the Batch 3 workspace
|
||||||
|
// model so the UI is forward-compatible with BooTerm/BooCoder.
|
||||||
|
export function NewPaneMenu({ onAddPane, disabled }: Props) {
|
||||||
|
return (
|
||||||
|
<DropdownMenu>
|
||||||
|
<DropdownMenuTrigger asChild>
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
disabled={disabled}
|
||||||
|
className="inline-flex items-center justify-center min-h-[44px] min-w-[44px] rounded-full bg-muted/40 hover:bg-muted/70 text-foreground disabled:opacity-40 disabled:cursor-not-allowed shrink-0"
|
||||||
|
aria-label="New pane"
|
||||||
|
>
|
||||||
|
<Plus size={16} />
|
||||||
|
</button>
|
||||||
|
</DropdownMenuTrigger>
|
||||||
|
<DropdownMenuContent align="end">
|
||||||
|
<DropdownMenuItem onSelect={() => onAddPane('chat')}>
|
||||||
|
<MessageSquare size={14} /> New chat
|
||||||
|
</DropdownMenuItem>
|
||||||
|
<DropdownMenuItem onSelect={() => onAddPane('terminal')}>
|
||||||
|
<Terminal size={14} /> New terminal
|
||||||
|
</DropdownMenuItem>
|
||||||
|
<DropdownMenuItem onSelect={() => onAddPane('agent')}>
|
||||||
|
<Bot size={14} /> New agent
|
||||||
|
</DropdownMenuItem>
|
||||||
|
</DropdownMenuContent>
|
||||||
|
</DropdownMenu>
|
||||||
|
);
|
||||||
|
}
|
||||||
36
apps/web/src/components/StatusDot.tsx
Normal file
36
apps/web/src/components/StatusDot.tsx
Normal file
@@ -0,0 +1,36 @@
|
|||||||
|
import { useChatStatus, type DerivedStatus } from '@/hooks/useChatStatus';
|
||||||
|
import { cn } from '@/lib/utils';
|
||||||
|
|
||||||
|
interface Props {
|
||||||
|
chatId: string | null | undefined;
|
||||||
|
className?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
const STATUS_CLASS: Record<DerivedStatus, string> = {
|
||||||
|
working: 'bg-amber-500 animate-pulse',
|
||||||
|
idle_warm: 'bg-emerald-500',
|
||||||
|
idle_cold: 'bg-muted-foreground/40',
|
||||||
|
error: 'bg-destructive',
|
||||||
|
};
|
||||||
|
|
||||||
|
const STATUS_LABEL: Record<DerivedStatus, string> = {
|
||||||
|
working: 'working',
|
||||||
|
idle_warm: 'idle',
|
||||||
|
idle_cold: 'idle',
|
||||||
|
error: 'error',
|
||||||
|
};
|
||||||
|
|
||||||
|
export function StatusDot({ chatId, className }: Props) {
|
||||||
|
const status = useChatStatus(chatId);
|
||||||
|
return (
|
||||||
|
<span
|
||||||
|
aria-label={`Status: ${STATUS_LABEL[status]}`}
|
||||||
|
title={STATUS_LABEL[status]}
|
||||||
|
className={cn(
|
||||||
|
'inline-block w-1.5 h-1.5 rounded-full shrink-0',
|
||||||
|
STATUS_CLASS[status],
|
||||||
|
className,
|
||||||
|
)}
|
||||||
|
/>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -1,14 +1,11 @@
|
|||||||
import { useCallback, useEffect } from 'react';
|
|
||||||
import { useLocation, useNavigate, useSearchParams } from 'react-router-dom';
|
|
||||||
import { PanelRight, MessageSquare, Terminal, Bot } from 'lucide-react';
|
import { PanelRight, MessageSquare, Terminal, Bot } from 'lucide-react';
|
||||||
import type { Chat, WorkspacePane } from '@/api/types';
|
import type { Chat, WorkspacePane } from '@/api/types';
|
||||||
import { useWorkspacePanes, MAX_PANES } from '@/hooks/useWorkspacePanes';
|
import { MAX_PANES, type UseWorkspacePanesResult } from '@/hooks/useWorkspacePanes';
|
||||||
import { useSessionChats } from '@/hooks/useSessionChats';
|
import type { UseSessionChatsResult } from '@/hooks/useSessionChats';
|
||||||
import { useViewport } from '@/hooks/useViewport';
|
import { useViewport } from '@/hooks/useViewport';
|
||||||
import { ChatPane } from '@/components/panes/ChatPane';
|
import { ChatPane } from '@/components/panes/ChatPane';
|
||||||
import { ChatTabBar } from '@/components/ChatTabBar';
|
import { ChatTabBar } from '@/components/ChatTabBar';
|
||||||
import { SessionLandingPage } from '@/components/SessionLandingPage';
|
import { SessionLandingPage } from '@/components/SessionLandingPage';
|
||||||
import { SwipeablePaneTab } from '@/components/SwipeablePaneTab';
|
|
||||||
import {
|
import {
|
||||||
DropdownMenu,
|
DropdownMenu,
|
||||||
DropdownMenuContent,
|
DropdownMenuContent,
|
||||||
@@ -20,14 +17,27 @@ import { cn } from '@/lib/utils';
|
|||||||
interface Props {
|
interface Props {
|
||||||
sessionId: string;
|
sessionId: string;
|
||||||
projectId: string;
|
projectId: string;
|
||||||
|
// Batch 9: threaded down to ChatPane → ChatInput → AgentPicker.
|
||||||
|
agentId?: string | null;
|
||||||
|
onAgentChange?: (agentId: string | null) => void | Promise<void>;
|
||||||
|
// v1.8: panes + chats hoisted into Session.tsx so the mobile header pill
|
||||||
|
// (MobileTabSwitcher) can share state with the pane grid.
|
||||||
|
panesHook: UseWorkspacePanesResult;
|
||||||
|
chatsHook: UseSessionChatsResult;
|
||||||
}
|
}
|
||||||
|
|
||||||
export function Workspace({ sessionId, projectId }: Props) {
|
export function Workspace({
|
||||||
|
sessionId,
|
||||||
|
projectId,
|
||||||
|
agentId,
|
||||||
|
onAgentChange,
|
||||||
|
panesHook,
|
||||||
|
chatsHook,
|
||||||
|
}: Props) {
|
||||||
const {
|
const {
|
||||||
panes,
|
panes,
|
||||||
activePaneIdx,
|
activePaneIdx,
|
||||||
setActivePaneIdx,
|
setActivePaneIdx,
|
||||||
activePaneIdxRef,
|
|
||||||
openChatInPane,
|
openChatInPane,
|
||||||
switchTab,
|
switchTab,
|
||||||
removeTab,
|
removeTab,
|
||||||
@@ -37,8 +47,6 @@ export function Workspace({ sessionId, projectId }: Props) {
|
|||||||
showLandingPage,
|
showLandingPage,
|
||||||
addSplitPane,
|
addSplitPane,
|
||||||
removePane,
|
removePane,
|
||||||
removeChatFromPanes,
|
|
||||||
initializeFirstChatIfEmpty,
|
|
||||||
handlePaneDragStart,
|
handlePaneDragStart,
|
||||||
handlePaneDragOver,
|
handlePaneDragOver,
|
||||||
handlePaneDragLeave,
|
handlePaneDragLeave,
|
||||||
@@ -46,15 +54,7 @@ export function Workspace({ sessionId, projectId }: Props) {
|
|||||||
handlePaneDragEnd,
|
handlePaneDragEnd,
|
||||||
dragOverIdx,
|
dragOverIdx,
|
||||||
draggingIdxRef,
|
draggingIdxRef,
|
||||||
} = useWorkspacePanes(sessionId);
|
} = panesHook;
|
||||||
|
|
||||||
// Thin wrapper so useSessionChats can route open_chat_in_active_pane events
|
|
||||||
// without knowing about pane indexing.
|
|
||||||
const openChatInActivePane = useCallback(
|
|
||||||
(chatId: string) => openChatInPane(activePaneIdxRef.current, chatId),
|
|
||||||
[openChatInPane, activePaneIdxRef],
|
|
||||||
);
|
|
||||||
|
|
||||||
const {
|
const {
|
||||||
chats,
|
chats,
|
||||||
createChat,
|
createChat,
|
||||||
@@ -63,47 +63,9 @@ export function Workspace({ sessionId, projectId }: Props) {
|
|||||||
deleteChat,
|
deleteChat,
|
||||||
renameChat,
|
renameChat,
|
||||||
handleLandingSend,
|
handleLandingSend,
|
||||||
} = useSessionChats(sessionId, {
|
} = chatsHook;
|
||||||
removeChatFromPanes,
|
|
||||||
openChatInPane,
|
|
||||||
openChatInActivePane,
|
|
||||||
initializeFirstChatIfEmpty,
|
|
||||||
});
|
|
||||||
|
|
||||||
const { isMobile } = useViewport();
|
const { isMobile } = useViewport();
|
||||||
const [searchParams] = useSearchParams();
|
|
||||||
const navigate = useNavigate();
|
|
||||||
const location = useLocation();
|
|
||||||
|
|
||||||
// URL -> state (mobile only). Handles deep-link arrival and Back button
|
|
||||||
// history pops. On a bare URL (no ?pane), reset to first pane so Back
|
|
||||||
// from a ?pane URL returns the user to a sensible default.
|
|
||||||
useEffect(() => {
|
|
||||||
if (!isMobile || panes.length === 0) return;
|
|
||||||
const paneId = searchParams.get('pane');
|
|
||||||
if (!paneId) {
|
|
||||||
if (activePaneIdx !== 0) setActivePaneIdx(0);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
const idx = panes.findIndex((p) => p.id === paneId);
|
|
||||||
if (idx >= 0 && idx !== activePaneIdx) setActivePaneIdx(idx);
|
|
||||||
}, [isMobile, searchParams, panes, activePaneIdx, setActivePaneIdx]);
|
|
||||||
|
|
||||||
// Switch active pane and push URL (mobile only). User-initiated only;
|
|
||||||
// never called from URL-sync effect.
|
|
||||||
const switchActivePane = useCallback(
|
|
||||||
(idx: number) => {
|
|
||||||
setActivePaneIdx(idx);
|
|
||||||
if (isMobile) {
|
|
||||||
const pane = panes[idx];
|
|
||||||
if (!pane) return;
|
|
||||||
const params = new URLSearchParams(location.search);
|
|
||||||
params.set('pane', pane.id);
|
|
||||||
navigate(`${location.pathname}?${params.toString()}`);
|
|
||||||
}
|
|
||||||
},
|
|
||||||
[setActivePaneIdx, isMobile, panes, navigate, location.pathname, location.search],
|
|
||||||
);
|
|
||||||
|
|
||||||
function chatsForPane(pane: WorkspacePane): Chat[] {
|
function chatsForPane(pane: WorkspacePane): Chat[] {
|
||||||
return pane.chatIds
|
return pane.chatIds
|
||||||
@@ -111,18 +73,6 @@ export function Workspace({ sessionId, projectId }: Props) {
|
|||||||
.filter((c): c is Chat => c !== undefined);
|
.filter((c): c is Chat => c !== undefined);
|
||||||
}
|
}
|
||||||
|
|
||||||
function paneLabel(pane: WorkspacePane): string {
|
|
||||||
const activeChatId = pane.chatId;
|
|
||||||
if (activeChatId) {
|
|
||||||
const chat = chats.find((c) => c.id === activeChatId);
|
|
||||||
if (chat) return chat.name ?? 'New chat';
|
|
||||||
}
|
|
||||||
if (pane.kind === 'chat') return 'Chat';
|
|
||||||
if (pane.kind === 'terminal') return 'Terminal';
|
|
||||||
if (pane.kind === 'agent') return 'Agent';
|
|
||||||
return 'Empty';
|
|
||||||
}
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="flex flex-col h-full min-h-0">
|
<div className="flex flex-col h-full min-h-0">
|
||||||
{!isMobile && (
|
{!isMobile && (
|
||||||
@@ -156,20 +106,8 @@ export function Workspace({ sessionId, projectId }: Props) {
|
|||||||
</div>
|
</div>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
{isMobile && panes.length > 1 && (
|
{/* v1.8: mobile multi-pane SwipeablePaneTab strip removed; the header
|
||||||
<div className="flex items-center gap-1 overflow-x-auto border-b border-border bg-muted/10 px-2 py-1 shrink-0">
|
pill (MobileTabSwitcher) is the mobile pane switcher. */}
|
||||||
{panes.map((pane, idx) => (
|
|
||||||
<SwipeablePaneTab
|
|
||||||
key={pane.id}
|
|
||||||
label={paneLabel(pane)}
|
|
||||||
isActive={idx === activePaneIdx}
|
|
||||||
onTap={() => switchActivePane(idx)}
|
|
||||||
onClose={() => removePane(idx)}
|
|
||||||
canClose={panes.length > 1}
|
|
||||||
/>
|
|
||||||
))}
|
|
||||||
</div>
|
|
||||||
)}
|
|
||||||
|
|
||||||
<div
|
<div
|
||||||
className={cn('flex-1 min-h-0', isMobile ? 'flex' : 'grid')}
|
className={cn('flex-1 min-h-0', isMobile ? 'flex' : 'grid')}
|
||||||
@@ -202,24 +140,36 @@ export function Workspace({ sessionId, projectId }: Props) {
|
|||||||
onDragStart={!isMobile && panes.length > 1 ? handlePaneDragStart(idx) : undefined}
|
onDragStart={!isMobile && panes.length > 1 ? handlePaneDragStart(idx) : undefined}
|
||||||
onDragEnd={!isMobile && panes.length > 1 ? handlePaneDragEnd : undefined}
|
onDragEnd={!isMobile && panes.length > 1 ? handlePaneDragEnd : undefined}
|
||||||
>
|
>
|
||||||
<ChatTabBar
|
{/* Hidden on mobile per v1.8: chat-within-pane navigation
|
||||||
pane={pane}
|
is not exposed on small screens; users switch panes via
|
||||||
tabs={chatsForPane(pane)}
|
the header pill instead. */}
|
||||||
onSwitchTab={(tabIdx) => switchTab(idx, tabIdx)}
|
{!isMobile && (
|
||||||
onRemoveTab={(chatId) => removeTab(idx, chatId)}
|
<ChatTabBar
|
||||||
onCloseOthers={(chatId) => closeOtherTabs(idx, chatId)}
|
pane={pane}
|
||||||
onCloseToRight={(chatId) => closeTabsToRight(idx, chatId)}
|
tabs={chatsForPane(pane)}
|
||||||
onCloseAll={() => closeAllTabs(idx)}
|
onSwitchTab={(tabIdx) => switchTab(idx, tabIdx)}
|
||||||
onNewChat={() => void createChat(idx)}
|
onRemoveTab={(chatId) => removeTab(idx, chatId)}
|
||||||
onShowHistory={() => showLandingPage(idx)}
|
onCloseOthers={(chatId) => closeOtherTabs(idx, chatId)}
|
||||||
onRename={renameChat}
|
onCloseToRight={(chatId) => closeTabsToRight(idx, chatId)}
|
||||||
onRemovePane={panes.length > 1 ? () => removePane(idx) : undefined}
|
onCloseAll={() => closeAllTabs(idx)}
|
||||||
/>
|
onNewChat={() => void createChat(idx)}
|
||||||
|
onShowHistory={() => showLandingPage(idx)}
|
||||||
|
onRename={renameChat}
|
||||||
|
onRemovePane={panes.length > 1 ? () => removePane(idx) : undefined}
|
||||||
|
/>
|
||||||
|
)}
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div className="flex-1 min-h-0 overflow-hidden">
|
<div className="flex-1 min-h-0 overflow-hidden">
|
||||||
{pane.kind === 'chat' && pane.chatId ? (
|
{pane.kind === 'chat' && pane.chatId ? (
|
||||||
<ChatPane sessionId={sessionId} chatId={pane.chatId} projectId={projectId} sessionChats={chats} />
|
<ChatPane
|
||||||
|
sessionId={sessionId}
|
||||||
|
chatId={pane.chatId}
|
||||||
|
projectId={projectId}
|
||||||
|
agentId={agentId}
|
||||||
|
onAgentChange={onAgentChange}
|
||||||
|
sessionChats={chats}
|
||||||
|
/>
|
||||||
) : (
|
) : (
|
||||||
<SessionLandingPage
|
<SessionLandingPage
|
||||||
sessionId={sessionId}
|
sessionId={sessionId}
|
||||||
|
|||||||
@@ -18,10 +18,13 @@ interface Props {
|
|||||||
sessionId: string;
|
sessionId: string;
|
||||||
chatId: string;
|
chatId: string;
|
||||||
projectId: string;
|
projectId: string;
|
||||||
|
// Batch 9: optional, threaded down to ChatInput's agent picker.
|
||||||
|
agentId?: string | null;
|
||||||
|
onAgentChange?: (agentId: string | null) => void | Promise<void>;
|
||||||
sessionChats?: import('@/api/types').Chat[];
|
sessionChats?: import('@/api/types').Chat[];
|
||||||
}
|
}
|
||||||
|
|
||||||
export function ChatPane({ sessionId, chatId, projectId, sessionChats }: Props) {
|
export function ChatPane({ sessionId, chatId, projectId, agentId, onAgentChange, sessionChats }: Props) {
|
||||||
const stream = useSessionStream(sessionId);
|
const stream = useSessionStream(sessionId);
|
||||||
const lastErrorRef = useRef<string | null>(null);
|
const lastErrorRef = useRef<string | null>(null);
|
||||||
const [queue, setQueue] = useState<string[]>([]);
|
const [queue, setQueue] = useState<string[]>([]);
|
||||||
@@ -167,7 +170,14 @@ export function ChatPane({ sessionId, chatId, projectId, sessionChats }: Props)
|
|||||||
|
|
||||||
<div className="relative">
|
<div className="relative">
|
||||||
<ChatContextPopover stats={contextStats} />
|
<ChatContextPopover stats={contextStats} />
|
||||||
<ChatInput disabled={false} projectId={projectId} onSend={handleSend} onForceSend={streaming ? handleForceSend : undefined} />
|
<ChatInput
|
||||||
|
disabled={false}
|
||||||
|
projectId={projectId}
|
||||||
|
agentId={agentId}
|
||||||
|
onAgentChange={onAgentChange}
|
||||||
|
onSend={handleSend}
|
||||||
|
onForceSend={streaming ? handleForceSend : undefined}
|
||||||
|
/>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
|
|||||||
@@ -115,6 +115,16 @@ export interface ProjectUpdatedEvent {
|
|||||||
name: string;
|
name: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// v1.8 mobile-tabs: broadcast on user channel from inference.ts so any device
|
||||||
|
// subscribed sees a chat working/idle/error. Frontend stores per-chat; panes
|
||||||
|
// derive their dot from pane.activeChatId.
|
||||||
|
export interface ChatStatusEvent {
|
||||||
|
type: 'chat_status';
|
||||||
|
chat_id: string;
|
||||||
|
status: 'working' | 'idle' | 'error';
|
||||||
|
at: string;
|
||||||
|
}
|
||||||
|
|
||||||
export type SessionEvent =
|
export type SessionEvent =
|
||||||
| SessionRenamedEvent
|
| SessionRenamedEvent
|
||||||
| ProjectCreatedEvent
|
| ProjectCreatedEvent
|
||||||
@@ -134,7 +144,8 @@ export type SessionEvent =
|
|||||||
| ChatDeletedEvent
|
| ChatDeletedEvent
|
||||||
| ProjectArchivedEvent
|
| ProjectArchivedEvent
|
||||||
| ProjectUnarchivedEvent
|
| ProjectUnarchivedEvent
|
||||||
| ProjectUpdatedEvent;
|
| ProjectUpdatedEvent
|
||||||
|
| ChatStatusEvent;
|
||||||
type Listener = (event: SessionEvent) => void;
|
type Listener = (event: SessionEvent) => void;
|
||||||
|
|
||||||
const listeners = new Set<Listener>();
|
const listeners = new Set<Listener>();
|
||||||
|
|||||||
71
apps/web/src/hooks/useChatStatus.ts
Normal file
71
apps/web/src/hooks/useChatStatus.ts
Normal file
@@ -0,0 +1,71 @@
|
|||||||
|
import { useEffect, useState } from 'react';
|
||||||
|
import { sessionEvents } from './sessionEvents';
|
||||||
|
|
||||||
|
export type RawStatus = 'working' | 'idle' | 'error';
|
||||||
|
export type DerivedStatus = 'working' | 'idle_warm' | 'idle_cold' | 'error';
|
||||||
|
|
||||||
|
// Window during which an idle dot stays green; after this, it fades to gray.
|
||||||
|
const WARM_WINDOW_MS = 30_000;
|
||||||
|
const TICK_MS = 5_000;
|
||||||
|
|
||||||
|
interface Entry {
|
||||||
|
status: RawStatus;
|
||||||
|
at: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Module-scope shared state so every StatusDot in the app shares one map
|
||||||
|
// (mirrors useSidebar's singleton pattern). The map is ephemeral — cleared on
|
||||||
|
// page reload; WS reconnect repopulates as new frames arrive.
|
||||||
|
const statuses = new Map<string, Entry>();
|
||||||
|
const subscribers = new Set<() => void>();
|
||||||
|
|
||||||
|
function notify(): void {
|
||||||
|
for (const s of subscribers) {
|
||||||
|
try { s(); } catch { /* swallow */ }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Guard against duplicate listeners during Vite HMR.
|
||||||
|
const G = globalThis as Record<string, unknown>;
|
||||||
|
if (!G.__boocode_chat_status_subscribed) {
|
||||||
|
G.__boocode_chat_status_subscribed = true;
|
||||||
|
sessionEvents.subscribe((ev) => {
|
||||||
|
if (ev.type !== 'chat_status') return;
|
||||||
|
statuses.set(ev.chat_id, { status: ev.status, at: ev.at });
|
||||||
|
notify();
|
||||||
|
});
|
||||||
|
// Single shared ticker: re-notify so any green dot whose 30s window just
|
||||||
|
// expired re-renders as gray. We only notify if there's something warm —
|
||||||
|
// avoids waking sleeping components for nothing.
|
||||||
|
setInterval(() => {
|
||||||
|
const now = Date.now();
|
||||||
|
for (const entry of statuses.values()) {
|
||||||
|
if (entry.status === 'idle') {
|
||||||
|
const age = now - new Date(entry.at).getTime();
|
||||||
|
if (age < WARM_WINDOW_MS + TICK_MS) {
|
||||||
|
notify();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}, TICK_MS);
|
||||||
|
}
|
||||||
|
|
||||||
|
function derive(entry: Entry | undefined): DerivedStatus {
|
||||||
|
if (!entry) return 'idle_cold';
|
||||||
|
if (entry.status === 'working') return 'working';
|
||||||
|
if (entry.status === 'error') return 'error';
|
||||||
|
const age = Date.now() - new Date(entry.at).getTime();
|
||||||
|
return age < WARM_WINDOW_MS ? 'idle_warm' : 'idle_cold';
|
||||||
|
}
|
||||||
|
|
||||||
|
export function useChatStatus(chatId: string | null | undefined): DerivedStatus {
|
||||||
|
const [, force] = useState({});
|
||||||
|
useEffect(() => {
|
||||||
|
const sub = () => force({});
|
||||||
|
subscribers.add(sub);
|
||||||
|
return () => { subscribers.delete(sub); };
|
||||||
|
}, []);
|
||||||
|
if (!chatId) return 'idle_cold';
|
||||||
|
return derive(statuses.get(chatId));
|
||||||
|
}
|
||||||
41
apps/web/src/hooks/useProjectGit.ts
Normal file
41
apps/web/src/hooks/useProjectGit.ts
Normal file
@@ -0,0 +1,41 @@
|
|||||||
|
import { useEffect, useState } from 'react';
|
||||||
|
import { api } from '@/api/client';
|
||||||
|
import type { GitMeta } from '@/api/types';
|
||||||
|
|
||||||
|
const POLL_INTERVAL_MS = 30_000;
|
||||||
|
|
||||||
|
// Live-ish git meta for the project header indicator. Backed by the server's
|
||||||
|
// 30s cache, so a 30s client poll plus the cache TTL bounds total staleness
|
||||||
|
// to ~60s in the worst case. Returns null while the first fetch is in flight
|
||||||
|
// or if the request failed.
|
||||||
|
export function useProjectGit(projectId: string | null | undefined): GitMeta | null {
|
||||||
|
const [meta, setMeta] = useState<GitMeta | null>(null);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
if (!projectId) {
|
||||||
|
setMeta(null);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
let cancelled = false;
|
||||||
|
|
||||||
|
const fetchOnce = () => {
|
||||||
|
api.projects
|
||||||
|
.git(projectId)
|
||||||
|
.then((m) => {
|
||||||
|
if (!cancelled) setMeta(m);
|
||||||
|
})
|
||||||
|
.catch(() => {
|
||||||
|
if (!cancelled) setMeta(null);
|
||||||
|
});
|
||||||
|
};
|
||||||
|
|
||||||
|
fetchOnce();
|
||||||
|
const t = setInterval(fetchOnce, POLL_INTERVAL_MS);
|
||||||
|
return () => {
|
||||||
|
cancelled = true;
|
||||||
|
clearInterval(t);
|
||||||
|
};
|
||||||
|
}, [projectId]);
|
||||||
|
|
||||||
|
return meta;
|
||||||
|
}
|
||||||
@@ -171,6 +171,7 @@ function applyEvent(prev: SidebarResponse, event: import('./sessionEvents').Sess
|
|||||||
case 'chat_archived':
|
case 'chat_archived':
|
||||||
case 'chat_unarchived':
|
case 'chat_unarchived':
|
||||||
case 'chat_deleted':
|
case 'chat_deleted':
|
||||||
|
case 'chat_status':
|
||||||
return prev;
|
return prev;
|
||||||
case 'project_archived': {
|
case 'project_archived': {
|
||||||
const next = prev.projects.filter((p) => p.id !== event.project_id);
|
const next = prev.projects.filter((p) => p.id !== event.project_id);
|
||||||
|
|||||||
@@ -1,5 +1,11 @@
|
|||||||
import { useEffect, useState } from 'react';
|
import { useCallback, useEffect, useState } from 'react';
|
||||||
import { Link, useNavigate, useParams } from 'react-router-dom';
|
import {
|
||||||
|
Link,
|
||||||
|
useLocation,
|
||||||
|
useNavigate,
|
||||||
|
useParams,
|
||||||
|
useSearchParams,
|
||||||
|
} from 'react-router-dom';
|
||||||
import { ChevronRight, FolderTree, Menu } from 'lucide-react';
|
import { ChevronRight, FolderTree, Menu } from 'lucide-react';
|
||||||
import { api } from '@/api/client';
|
import { api } from '@/api/client';
|
||||||
import type { Project, Session as SessionType } from '@/api/types';
|
import type { Project, Session as SessionType } from '@/api/types';
|
||||||
@@ -8,12 +14,28 @@ import { useActivePane } from '@/hooks/useActivePane';
|
|||||||
import { useSidebarDrawer } from '@/hooks/useSidebarDrawer';
|
import { useSidebarDrawer } from '@/hooks/useSidebarDrawer';
|
||||||
import { useRightRailDrawer } from '@/hooks/useRightRailDrawer';
|
import { useRightRailDrawer } from '@/hooks/useRightRailDrawer';
|
||||||
import { useViewport } from '@/hooks/useViewport';
|
import { useViewport } from '@/hooks/useViewport';
|
||||||
|
import { useWorkspacePanes, MAX_PANES } from '@/hooks/useWorkspacePanes';
|
||||||
|
import { useSessionChats } from '@/hooks/useSessionChats';
|
||||||
|
import { useProjectGit } from '@/hooks/useProjectGit';
|
||||||
import { Workspace } from '@/components/Workspace';
|
import { Workspace } from '@/components/Workspace';
|
||||||
import { ModelPicker } from '@/components/ModelPicker';
|
import { ModelPicker } from '@/components/ModelPicker';
|
||||||
|
import { MobileTabSwitcher } from '@/components/MobileTabSwitcher';
|
||||||
|
import { NewPaneMenu } from '@/components/NewPaneMenu';
|
||||||
|
import { cn } from '@/lib/utils';
|
||||||
|
|
||||||
export function Session() {
|
export function Session() {
|
||||||
const { id } = useParams<{ id: string }>();
|
const { id } = useParams<{ id: string }>();
|
||||||
|
if (!id) return null;
|
||||||
|
// v1.8: key on id so route navigation remounts SessionInner — the hoisted
|
||||||
|
// useWorkspacePanes + useSessionChats then reinitialize cleanly from the
|
||||||
|
// new sessionId instead of carrying stale state across sessions.
|
||||||
|
return <SessionInner key={id} sessionId={id} />;
|
||||||
|
}
|
||||||
|
|
||||||
|
function SessionInner({ sessionId }: { sessionId: string }) {
|
||||||
const navigate = useNavigate();
|
const navigate = useNavigate();
|
||||||
|
const location = useLocation();
|
||||||
|
const [searchParams] = useSearchParams();
|
||||||
const [session, setSession] = useState<SessionType | null>(null);
|
const [session, setSession] = useState<SessionType | null>(null);
|
||||||
const [project, setProject] = useState<Project | null>(null);
|
const [project, setProject] = useState<Project | null>(null);
|
||||||
const [name, setName] = useState('');
|
const [name, setName] = useState('');
|
||||||
@@ -23,23 +45,53 @@ export function Session() {
|
|||||||
const { toggle: toggleRightRail } = useRightRailDrawer();
|
const { toggle: toggleRightRail } = useRightRailDrawer();
|
||||||
const { isMobile } = useViewport();
|
const { isMobile } = useViewport();
|
||||||
|
|
||||||
|
// v1.8: pane + chat state hoisted into Session so the mobile header pill
|
||||||
|
// (MobileTabSwitcher) shares one source of truth with the pane grid below.
|
||||||
|
const panesHook = useWorkspacePanes(sessionId);
|
||||||
|
const {
|
||||||
|
panes,
|
||||||
|
activePaneIdx,
|
||||||
|
setActivePaneIdx,
|
||||||
|
openChatInPane,
|
||||||
|
activePaneIdxRef,
|
||||||
|
addSplitPane,
|
||||||
|
removePane,
|
||||||
|
removeChatFromPanes,
|
||||||
|
initializeFirstChatIfEmpty,
|
||||||
|
} = panesHook;
|
||||||
|
|
||||||
|
const openChatInActivePane = useCallback(
|
||||||
|
(chatId: string) => openChatInPane(activePaneIdxRef.current, chatId),
|
||||||
|
[openChatInPane, activePaneIdxRef],
|
||||||
|
);
|
||||||
|
const chatsHook = useSessionChats(sessionId, {
|
||||||
|
removeChatFromPanes,
|
||||||
|
openChatInPane,
|
||||||
|
openChatInActivePane,
|
||||||
|
initializeFirstChatIfEmpty,
|
||||||
|
});
|
||||||
|
const { chats, renameChat } = chatsHook;
|
||||||
|
|
||||||
|
// v1.8 Level 1: branch indicator. Polls every 30s; server caches the same
|
||||||
|
// span so back-to-back loads are cheap. Returns null until the first fetch
|
||||||
|
// resolves or if the project isn't a git repo.
|
||||||
|
const git = useProjectGit(project?.id);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
if (!id) return;
|
|
||||||
setSession(null);
|
setSession(null);
|
||||||
setProject(null);
|
setProject(null);
|
||||||
let cancelled = false;
|
let cancelled = false;
|
||||||
api.sessions
|
api.sessions
|
||||||
.get(id)
|
.get(sessionId)
|
||||||
.then((s) => {
|
.then((s) => {
|
||||||
if (cancelled) return;
|
if (cancelled) return;
|
||||||
setSession(s);
|
setSession(s);
|
||||||
setName(s.name);
|
setName(s.name);
|
||||||
sessionEvents.emit({
|
sessionEvents.emit({
|
||||||
type: 'session_loaded',
|
type: 'session_loaded',
|
||||||
session_id: id,
|
session_id: sessionId,
|
||||||
project_id: s.project_id,
|
project_id: s.project_id,
|
||||||
});
|
});
|
||||||
// Load project for breadcrumb. Listing is fine — small N, cached by client.
|
|
||||||
api.projects.list().then((projects) => {
|
api.projects.list().then((projects) => {
|
||||||
if (cancelled) return;
|
if (cancelled) return;
|
||||||
const p = projects.find((x) => x.id === s.project_id);
|
const p = projects.find((x) => x.id === s.project_id);
|
||||||
@@ -50,34 +102,61 @@ export function Session() {
|
|||||||
return () => {
|
return () => {
|
||||||
cancelled = true;
|
cancelled = true;
|
||||||
};
|
};
|
||||||
}, [id]);
|
}, [sessionId]);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
if (!id) return;
|
|
||||||
return sessionEvents.subscribe((event) => {
|
return sessionEvents.subscribe((event) => {
|
||||||
if (event.type === 'session_renamed' && event.session_id === id) {
|
if (event.type === 'session_renamed' && event.session_id === sessionId) {
|
||||||
setSession((prev) => (prev ? { ...prev, name: event.name } : prev));
|
setSession((prev) => (prev ? { ...prev, name: event.name } : prev));
|
||||||
setName((prev) => (editingName ? prev : event.name));
|
setName((prev) => (editingName ? prev : event.name));
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
if (
|
if (
|
||||||
(event.type === 'session_deleted' || event.type === 'session_archived') &&
|
(event.type === 'session_deleted' || event.type === 'session_archived') &&
|
||||||
event.session_id === id
|
event.session_id === sessionId
|
||||||
) {
|
) {
|
||||||
navigate(`/project/${event.project_id}`);
|
navigate(`/project/${event.project_id}`);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
}, [id, editingName, navigate]);
|
}, [sessionId, editingName, navigate]);
|
||||||
|
|
||||||
|
// v1.8: URL ?pane= sync (mobile only). Lifted from Workspace.tsx so
|
||||||
|
// MobileTabSwitcher's onSwitchPane can push the same URL state and the
|
||||||
|
// browser Back button continues to walk pane history on mobile.
|
||||||
|
useEffect(() => {
|
||||||
|
if (!isMobile || panes.length === 0) return;
|
||||||
|
const paneId = searchParams.get('pane');
|
||||||
|
if (!paneId) {
|
||||||
|
if (activePaneIdx !== 0) setActivePaneIdx(0);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
const idx = panes.findIndex((p) => p.id === paneId);
|
||||||
|
if (idx >= 0 && idx !== activePaneIdx) setActivePaneIdx(idx);
|
||||||
|
}, [isMobile, searchParams, panes, activePaneIdx, setActivePaneIdx]);
|
||||||
|
|
||||||
|
const switchActivePane = useCallback(
|
||||||
|
(idx: number) => {
|
||||||
|
setActivePaneIdx(idx);
|
||||||
|
if (isMobile) {
|
||||||
|
const pane = panes[idx];
|
||||||
|
if (!pane) return;
|
||||||
|
const params = new URLSearchParams(location.search);
|
||||||
|
params.set('pane', pane.id);
|
||||||
|
navigate(`${location.pathname}?${params.toString()}`);
|
||||||
|
}
|
||||||
|
},
|
||||||
|
[setActivePaneIdx, isMobile, panes, navigate, location.pathname, location.search],
|
||||||
|
);
|
||||||
|
|
||||||
async function saveName() {
|
async function saveName() {
|
||||||
if (!id || !session) return;
|
if (!session) return;
|
||||||
const trimmed = name.trim();
|
const trimmed = name.trim();
|
||||||
if (!trimmed || trimmed === session.name) {
|
if (!trimmed || trimmed === session.name) {
|
||||||
setName(session.name);
|
setName(session.name);
|
||||||
setEditingName(false);
|
setEditingName(false);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
const updated = await api.sessions.update(id, { name: trimmed });
|
const updated = await api.sessions.update(sessionId, { name: trimmed });
|
||||||
setSession(updated);
|
setSession(updated);
|
||||||
setEditingName(false);
|
setEditingName(false);
|
||||||
// Server publishes session_renamed via broker.publishUser; no local emit needed.
|
// Server publishes session_renamed via broker.publishUser; no local emit needed.
|
||||||
@@ -85,115 +164,180 @@ export function Session() {
|
|||||||
|
|
||||||
// Workspace only sets activeFile for file-browser panes; checking it alone
|
// Workspace only sets activeFile for file-browser panes; checking it alone
|
||||||
// suffices and is forward-compatible with future pane kinds.
|
// suffices and is forward-compatible with future pane kinds.
|
||||||
const showActiveFile = active.sessionId === id && !!active.activeFile;
|
const showActiveFile = active.sessionId === sessionId && !!active.activeFile;
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="flex-1 flex flex-col min-h-0">
|
<div className="flex-1 flex flex-col min-h-0">
|
||||||
<header
|
<header
|
||||||
className="border-b px-3 sm:px-4 py-2 flex items-center gap-1.5 shrink-0 text-sm"
|
className={cn(
|
||||||
|
'border-b shrink-0 text-sm',
|
||||||
|
isMobile
|
||||||
|
? 'flex flex-col gap-1.5 px-3 py-2'
|
||||||
|
: 'flex items-center gap-1.5 px-3 sm:px-4 py-2',
|
||||||
|
)}
|
||||||
style={{ paddingTop: 'max(0.5rem, env(safe-area-inset-top))' }}
|
style={{ paddingTop: 'max(0.5rem, env(safe-area-inset-top))' }}
|
||||||
>
|
>
|
||||||
{isMobile && (
|
{isMobile ? (
|
||||||
<button
|
|
||||||
type="button"
|
|
||||||
onClick={() => setDrawerOpen(true)}
|
|
||||||
className="inline-flex items-center justify-center -ml-1 min-w-[44px] min-h-[44px] rounded text-muted-foreground hover:bg-muted hover:text-foreground shrink-0"
|
|
||||||
aria-label="Open sidebar"
|
|
||||||
>
|
|
||||||
<Menu className="size-5" />
|
|
||||||
</button>
|
|
||||||
)}
|
|
||||||
|
|
||||||
{/* Breadcrumb — desktop only */}
|
|
||||||
<div className="hidden sm:flex items-center gap-1.5 min-w-0">
|
|
||||||
<Link to="/" className="text-muted-foreground hover:text-foreground shrink-0 text-xs">
|
|
||||||
Projects
|
|
||||||
</Link>
|
|
||||||
<ChevronRight className="size-3 text-muted-foreground/60 shrink-0" />
|
|
||||||
{project ? (
|
|
||||||
<Link
|
|
||||||
to={`/project/${project.id}`}
|
|
||||||
className="text-muted-foreground hover:text-foreground truncate max-w-[200px]"
|
|
||||||
title={project.name}
|
|
||||||
>
|
|
||||||
{project.name}
|
|
||||||
</Link>
|
|
||||||
) : (
|
|
||||||
<span className="text-muted-foreground/60">…</span>
|
|
||||||
)}
|
|
||||||
<ChevronRight className="size-3 text-muted-foreground/60 shrink-0" />
|
|
||||||
</div>
|
|
||||||
|
|
||||||
{/* Session name — always visible, truncated, editable */}
|
|
||||||
{editingName ? (
|
|
||||||
<input
|
|
||||||
autoFocus
|
|
||||||
value={name}
|
|
||||||
onChange={(e) => setName(e.target.value)}
|
|
||||||
onBlur={() => void saveName()}
|
|
||||||
onKeyDown={(e) => {
|
|
||||||
if (e.key === 'Enter') void saveName();
|
|
||||||
if (e.key === 'Escape') {
|
|
||||||
setName(session?.name ?? '');
|
|
||||||
setEditingName(false);
|
|
||||||
}
|
|
||||||
}}
|
|
||||||
className="bg-transparent border-b border-border px-1 py-0.5 text-sm font-medium outline-none focus:border-ring min-w-0"
|
|
||||||
/>
|
|
||||||
) : (
|
|
||||||
<button
|
|
||||||
type="button"
|
|
||||||
className="text-sm font-medium hover:underline truncate max-w-[140px] sm:max-w-[280px] min-w-0"
|
|
||||||
onClick={() => setEditingName(true)}
|
|
||||||
title={session?.name ?? ''}
|
|
||||||
>
|
|
||||||
{session?.name ?? '…'}
|
|
||||||
</button>
|
|
||||||
)}
|
|
||||||
|
|
||||||
{/* Active file — desktop only */}
|
|
||||||
{showActiveFile && active.activeFile && (
|
|
||||||
<>
|
<>
|
||||||
<span className="text-muted-foreground/40 mx-1 hidden sm:inline">·</span>
|
{/* v1.8 mobile row 1: hamburger | repo+branch | ModelPicker | FolderTree.
|
||||||
<span
|
Gear/kebab cluster lands in Batch 7; ModelPicker stays here until
|
||||||
className="text-xs font-mono text-muted-foreground truncate max-w-[200px] hidden sm:inline"
|
then so mobile users keep model-switching access. */}
|
||||||
title={active.activeFile}
|
<div className="flex items-center gap-1.5">
|
||||||
>
|
<button
|
||||||
{active.activeFile}
|
type="button"
|
||||||
</span>
|
onClick={() => setDrawerOpen(true)}
|
||||||
</>
|
className="inline-flex items-center justify-center -ml-1 min-w-[44px] min-h-[44px] rounded text-muted-foreground hover:bg-muted hover:text-foreground shrink-0"
|
||||||
)}
|
aria-label="Open sidebar"
|
||||||
|
>
|
||||||
|
<Menu className="size-5" />
|
||||||
|
</button>
|
||||||
|
|
||||||
{/* Model picker — right-aligned */}
|
<div className="flex-1 min-w-0 flex items-center justify-center gap-1.5">
|
||||||
<div className="ml-auto shrink-0">
|
{project ? (
|
||||||
{session && (
|
<span className="text-sm font-medium truncate" title={project.name}>
|
||||||
<div className="inline-flex items-center rounded-full bg-muted/40 hover:bg-muted/70 px-1">
|
{project.name}
|
||||||
<ModelPicker
|
</span>
|
||||||
value={session.model}
|
) : (
|
||||||
onChange={async (model) => {
|
<span className="text-muted-foreground/60">…</span>
|
||||||
const updated = await api.sessions.update(session.id, { model });
|
)}
|
||||||
setSession(updated);
|
{git?.branch && (
|
||||||
}}
|
<span
|
||||||
|
className="text-muted-foreground/80 text-xs truncate"
|
||||||
|
title={`branch: ${git.branch}${git.is_dirty ? ' (dirty)' : ''}`}
|
||||||
|
>
|
||||||
|
· {git.branch}
|
||||||
|
</span>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{session && (
|
||||||
|
<div className="inline-flex items-center rounded-full bg-muted/40 hover:bg-muted/70 px-1 shrink-0">
|
||||||
|
<ModelPicker
|
||||||
|
value={session.model}
|
||||||
|
onChange={async (model) => {
|
||||||
|
const updated = await api.sessions.update(session.id, { model });
|
||||||
|
setSession(updated);
|
||||||
|
}}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
|
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
onClick={toggleRightRail}
|
||||||
|
className="inline-flex items-center justify-center -mr-1 min-w-[44px] min-h-[44px] rounded text-muted-foreground hover:bg-muted hover:text-foreground shrink-0"
|
||||||
|
aria-label="Toggle file browser"
|
||||||
|
>
|
||||||
|
<FolderTree className="size-5" />
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{/* v1.8 mobile row 2: pane-switcher pill + new-pane menu. Pill
|
||||||
|
expands; NewPaneMenu is the trailing 44x44 trigger. */}
|
||||||
|
<div className="flex items-center gap-1.5">
|
||||||
|
<MobileTabSwitcher
|
||||||
|
panes={panes}
|
||||||
|
activePaneIdx={activePaneIdx}
|
||||||
|
chats={chats}
|
||||||
|
onSwitchPane={switchActivePane}
|
||||||
|
onRemovePane={removePane}
|
||||||
|
onRenameChat={renameChat}
|
||||||
|
/>
|
||||||
|
<NewPaneMenu
|
||||||
|
onAddPane={addSplitPane}
|
||||||
|
disabled={panes.length >= MAX_PANES}
|
||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
)}
|
</>
|
||||||
</div>
|
) : (
|
||||||
|
<>
|
||||||
|
{/* Desktop: unchanged single-row header. */}
|
||||||
|
<div className="hidden sm:flex items-center gap-1.5 min-w-0">
|
||||||
|
<Link to="/" className="text-muted-foreground hover:text-foreground shrink-0 text-xs">
|
||||||
|
Projects
|
||||||
|
</Link>
|
||||||
|
<ChevronRight className="size-3 text-muted-foreground/60 shrink-0" />
|
||||||
|
{project ? (
|
||||||
|
<Link
|
||||||
|
to={`/project/${project.id}`}
|
||||||
|
className="text-muted-foreground hover:text-foreground truncate max-w-[200px]"
|
||||||
|
title={project.name}
|
||||||
|
>
|
||||||
|
{project.name}
|
||||||
|
</Link>
|
||||||
|
) : (
|
||||||
|
<span className="text-muted-foreground/60">…</span>
|
||||||
|
)}
|
||||||
|
<ChevronRight className="size-3 text-muted-foreground/60 shrink-0" />
|
||||||
|
</div>
|
||||||
|
|
||||||
{/* File browser toggle — mobile only */}
|
{editingName ? (
|
||||||
{isMobile && (
|
<input
|
||||||
<button
|
autoFocus
|
||||||
type="button"
|
value={name}
|
||||||
onClick={toggleRightRail}
|
onChange={(e) => setName(e.target.value)}
|
||||||
className="inline-flex items-center justify-center -mr-1 min-w-[44px] min-h-[44px] rounded text-muted-foreground hover:bg-muted hover:text-foreground shrink-0"
|
onBlur={() => void saveName()}
|
||||||
aria-label="Toggle file browser"
|
onKeyDown={(e) => {
|
||||||
>
|
if (e.key === 'Enter') void saveName();
|
||||||
<FolderTree className="size-5" />
|
if (e.key === 'Escape') {
|
||||||
</button>
|
setName(session?.name ?? '');
|
||||||
|
setEditingName(false);
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
className="bg-transparent border-b border-border px-1 py-0.5 text-sm font-medium outline-none focus:border-ring min-w-0"
|
||||||
|
/>
|
||||||
|
) : (
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
className="text-sm font-medium hover:underline truncate max-w-[280px] min-w-0"
|
||||||
|
onClick={() => setEditingName(true)}
|
||||||
|
title={session?.name ?? ''}
|
||||||
|
>
|
||||||
|
{session?.name ?? '…'}
|
||||||
|
</button>
|
||||||
|
)}
|
||||||
|
|
||||||
|
{showActiveFile && active.activeFile && (
|
||||||
|
<>
|
||||||
|
<span className="text-muted-foreground/40 mx-1">·</span>
|
||||||
|
<span
|
||||||
|
className="text-xs font-mono text-muted-foreground truncate max-w-[200px]"
|
||||||
|
title={active.activeFile}
|
||||||
|
>
|
||||||
|
{active.activeFile}
|
||||||
|
</span>
|
||||||
|
</>
|
||||||
|
)}
|
||||||
|
|
||||||
|
<div className="ml-auto shrink-0">
|
||||||
|
{session && (
|
||||||
|
<div className="inline-flex items-center rounded-full bg-muted/40 hover:bg-muted/70 px-1">
|
||||||
|
<ModelPicker
|
||||||
|
value={session.model}
|
||||||
|
onChange={async (model) => {
|
||||||
|
const updated = await api.sessions.update(session.id, { model });
|
||||||
|
setSession(updated);
|
||||||
|
}}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
</>
|
||||||
)}
|
)}
|
||||||
</header>
|
</header>
|
||||||
|
|
||||||
{id && session && (
|
{session && (
|
||||||
<Workspace sessionId={id} projectId={session.project_id} />
|
<Workspace
|
||||||
|
sessionId={sessionId}
|
||||||
|
projectId={session.project_id}
|
||||||
|
agentId={session.agent_id}
|
||||||
|
onAgentChange={async (agent_id) => {
|
||||||
|
const updated = await api.sessions.update(session.id, { agent_id });
|
||||||
|
setSession(updated);
|
||||||
|
}}
|
||||||
|
panesHook={panesHook}
|
||||||
|
chatsHook={chatsHook}
|
||||||
|
/>
|
||||||
)}
|
)}
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
|
|||||||
@@ -4,512 +4,337 @@ Last updated: 2026-05-16
|
|||||||
|
|
||||||
## Overview
|
## Overview
|
||||||
|
|
||||||
BooCode is a standalone code-chat tool at `/opt/boocode/`. Read-only by design in v1.x — pick a project, chat with a local LLM that has file-inspection tools, get streaming responses over WebSocket. Built May 2026 after the in-boolab BooCode mode stalled.
|
BooCode is a standalone code-chat tool at `/opt/boocode/`. Read-only by design — pick a project, chat with a local LLM that has file-inspection tools, get streaming responses over WebSocket.
|
||||||
|
|
||||||
v1 shipped in a single Claude Code session. v1.1 onwards is a batched build-out. Original Batch 1–10 plan was reordered mid-stream — chats-inside-sessions, archive, and fork/delete work was prioritized over the mobile pass.
|
|
||||||
|
|
||||||
Live at `https://code.indifferentketchup.com` (Caddy → Authelia → Tailscale → `100.114.205.53:9500`).
|
Live at `https://code.indifferentketchup.com` (Caddy → Authelia → Tailscale → `100.114.205.53:9500`).
|
||||||
|
|
||||||
-----
|
**Architectural commitments:**
|
||||||
|
|
||||||
## Version summary
|
- No embeddings. The model uses file-view tools (`view_file`, `list_dir`, `grep`, `find_files`) + sidecar analyzers (codecontext, codesight). Walked away from the RAG pipeline May 2026.
|
||||||
|
- Read-only in v1.x. Write tools land in BooCoder (separate container, post-v1.x).
|
||||||
|
- One Postgres (`boocode_db`), one frontend SPA, container-per-service for new capabilities.
|
||||||
|
|
||||||
|Version |Theme |Status |Notes |
|
External code lifted from / referenced in: see `boocode_code_review.md` for full inventory.
|
||||||
|----------------|---------------------------------------------------------------------------------------------------------------------------------------|---------------------------------|-----------------------------------------|
|
|
||||||
|v1.0 |Initial scaffold, read-only tools, WS streaming |✅ Done |Shipped in one Claude Code session |
|
|
||||||
|v1.1-batch1 |Markdown, Copy + Regen, tok/s + ctx, AI chat naming |✅ Merged |— |
|
|
||||||
|v1.1-batch2 |Sidebar restructure: projects → sessions, max 5 + “view all” |✅ Merged |— |
|
|
||||||
|v1.1-batch3 |Pane system, FileBrowserPane + Shiki, chat→file click, cross-tab |✅ Merged |— |
|
|
||||||
|v1.1-batch3.5 |Chip infrastructure, `@file` picker, line-select-attach |✅ Merged |— |
|
|
||||||
|v1.2 |Chats inside sessions refactor, right-rail, `/compact`, archive, force-send |✅ Merged |Replaced original “Batch 4 = mobile” plan|
|
|
||||||
|v1.2-project-ux |Project archive UX, sidebar context menu, full-bootstrap, Gitea API |✅ Merged |— |
|
|
||||||
|v1.3 |Tab-close + chat-archive |✅ Merged |— |
|
|
||||||
|v1.4 |Fork from message + delete message + header polish + housekeeping |✅ Merged |Was original “Batch 5” |
|
|
||||||
|v1.5 |Refactor splits, vitest harness (23 tests), error-log surfacing, `/opt:ro` + `BOOTSTRAP_ROOT`, persistent context-window tracker |✅ Merged |— |
|
|
||||||
|v1.5.1 |Bootstrap hotfix: git in container, SSH keypair, known_hosts, SSH URL rewrite, /opt/projects label |✅ Merged |`4a9f207` |
|
|
||||||
|v1.6-mobile-pass|Mobile pass: drawer, pane stacking, long-press, swipe-to-close, pull-to-refresh, IME safety, safe-area, tap targets + H1 path-guard fix|✅ Merged |`57c883b..943ae7d` (6 commits) |
|
|
||||||
|v1.6.1-cleanup |Mostly audit-only; one fix shipped: RightRail `max-md:hidden` wrapper. Audit reports for secrets, stale code, panes, mount scope, hand-rolled patterns deferred to follow-ups |✅ Merged |`6a9fe18` |
|
|
||||||
|v1.6.2-mobile-ui-fixes|Mobile UI polish from device testing: kill single-pane navigator chrome, header rework, “New chat” in long-press menu, RightRail as mobile drawer (reverts v1.6.1 wrapper) |🔄 Hand-back received, uncommitted|— |
|
|
||||||
|v1.7 |Drag-drop + paste-as-attachment (chip infra extension) |Planned |Was Batch 6 |
|
|
||||||
|v1.8 |Settings drawer (system prompt per project + session, web search toggle) |Planned |Was Batch 7 |
|
|
||||||
|v1.9 |Web search backend: SearXNG `web_search` + `web_fetch` tools |Planned |Was Batch 8 |
|
|
||||||
|v1.10 |Agents (Tier 2): `AGENTS.md`, per-agent model/temp/tools, picker |Planned |Was Batch 9 |
|
|
||||||
|v1.11 |BooTerm: separate container, xterm.js + node-pty + tmux, terminal pane |Planned |Was Batch 10 |
|
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
## Version details
|
## Batch summary
|
||||||
|
|
||||||
### v1.1-batch1 — Message polish ✅
|
|Batch |Theme |Status |Branch / Notes |
|
||||||
|
|------------------------------------------|-----------------------------------------------------------------------------------|-----------|---------------------------------------|
|
||||||
|
|1 |Markdown, Copy + Regen, tok/s + ctx, AI naming |✅ Done |`v1.1-batch1` merged |
|
||||||
|
|2 |Sidebar restructure |✅ Done |`v1.1-batch2` merged |
|
||||||
|
|3 |Pane system, FileBrowserPane + Shiki, cross-tab |✅ Done |`v1.1-batch3` merged |
|
||||||
|
|3.5 |Chip infrastructure, `@file`, line-select |✅ Done |merged |
|
||||||
|
|4 (v1.2) |Chats inside sessions, right-rail, `/compact`, archive, force-send |✅ Done |merged |
|
||||||
|
|4.1–4.4 |Project archive, sidebar context, Gitea API, bootstrap |✅ Done |merged |
|
||||||
|
|v1.5 cleanup |resolveProjectPath, BOOTSTRAP_ROOT, vitest pin |✅ Done |merged |
|
||||||
|
|v1.6 mobile |Drawer, single-pane, long-press, IME-safe, pull-to-refresh, swipe-close |✅ Done |merged |
|
||||||
|
|v1.6.1 |RightRail mobile wrapper fix |✅ Done |merged |
|
||||||
|
|Tool-loop bump |MAX_TOOL_LOOP_DEPTH 5→15 |✅ Done |merged |
|
||||||
|
|v1.6.2 |Workspace + Session+Project headers + ChatTabBar new-chat + RightRail mobile drawer|🔄 In flight|`v1.6.2-mobile-ui-fixes` |
|
||||||
|
|**v1.8 mobile tabs** |**Bottom-sheet pane switcher + cross-tab `pane_status` WS sync + StatusDot on tabs**|**Next up**|`v1.8-mobile-tabs`; hand-rolled sheet |
|
||||||
|
|9 (REORDERED, DECOUPLED) |Agents (Tier 2): `AGENTS.md`, per-agent temp/tools, picker in ChatInput toolbar |✅ Implemented, uncommitted|six builtins; on `main` awaiting commit|
|
||||||
|
|5 |Fork message, delete message, header polish |Planned | |
|
||||||
|
|6 |Drag-drop file + paste-as-attachment |Planned |thin extension of 3.5 chips |
|
||||||
|
|7 |Settings drawer: system prompt, web search toggle, agent entry |Planned |adds SettingsDrawer agent entry (Batch 9 deferred half) |
|
||||||
|
|8 |Web search backend: SearXNG `web_search` + `web_fetch` |Planned | |
|
||||||
|
|10 |BooTerm: separate container, xterm.js + node-pty + tmux |Planned | |
|
||||||
|
|11 — Architect: codebase map |codecontext sidecar + MCP tool wiring |Planned |from nmakod/codecontext |
|
||||||
|
|11b — Architect: repo health |call graph, circular deps, dead code |Planned |from spirituslab/codesight |
|
||||||
|
|12 — Tool approval + plan/act mode |Read-only invariant, per-tool gating |Planned |from cline |
|
||||||
|
|13 — Append-only event log |Replace messages-table semantics |Planned |from OpenHands V1 |
|
||||||
|
|14 — BooCoder: pending changes |Sandboxed edit queue, atomic apply |Post-v1.x |from plandex |
|
||||||
|
|15 — BooCoder runtime isolation |Per-session Docker sandbox |Post-v1.x |from OpenHands |
|
||||||
|
|16 — Multi-provider LLM |Optional litellm-style abstraction |Optional |from pi-ai |
|
||||||
|
|17 — Workflow graphs |Multi-agent coordination |Far future |from microsoft/agent-framework concepts|
|
||||||
|
|
||||||
Markdown (`react-markdown` + `remark-gfm`), Copy + Regenerate, tok/s + context counter, AI session naming.
|
**Old Batch 12 (codebase indexer w/ Harrier embeddings) — REMOVED.** Replaced by Batch 11/11b sidecar approach. See `boocode_code_review.md` decisions log.
|
||||||
|
|
||||||
**Key decisions:**
|
**Batch 9 reordered ahead of 5–8, 10.** Picker mounts in `ChatInput.tsx` toolbar only. SettingsDrawer agent entry rolled into Batch 7 when it lands. No UI dependency on Batches 5/6/7, so it can ship anytime after v1.6.2.
|
||||||
|
|
||||||
- `sessions.name` (not `title`).
|
|
||||||
- `enable_thinking: false` + `max_tokens: 30` for Qwen3 utility calls.
|
|
||||||
- `messages_deleted` WS frame added for multi-tab regen.
|
|
||||||
- In-app event bus (`sessionEvents.ts`, module-scope `Set<Listener>`).
|
|
||||||
|
|
||||||
**Schema:** `messages.tokens_used`, `messages.ctx_used`, `messages.ctx_max`, `messages.started_at`, `messages.finished_at`.
|
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.1-batch2 — Sidebar restructure ✅
|
## Batch details (planned / new)
|
||||||
|
|
||||||
Projects as expandable groups, up to 5 recent sessions per project, “View all (N)”, `GET /api/sidebar`, `useSidebar` singleton hook.
|
### Batch 9 — Agents (Tier 2, DECOUPLED)
|
||||||
|
|
||||||
**Key decisions:**
|
**Spec:** `boocode_batch9.md` with the deltas below.
|
||||||
|
|
||||||
- `useSidebar` module-scope singleton.
|
**Status:** Next up after v1.6.2 merges. Decoupled from Batch 7.
|
||||||
- `localStorage['boocode.sidebar.expanded']`.
|
|
||||||
- `session_renamed` payload `{session_id, name}`.
|
**Deltas from `boocode_batch9.md`:**
|
||||||
|
|
||||||
|
1. Builtin defaults in `agents.ts` OMIT the `model` field. Resolution order makes `session.model` win when `agent.model` is null. Spec line 30 example is misleading — do not hardcode any model in builtins.
|
||||||
|
2. Builtin defaults are the six agents shipped in `/opt/boocode/AGENTS.md`: **Code Reviewer, Debugger, Refactorer, Architect, Security Auditor, Prompt Builder.** If project root `AGENTS.md` exists, only its agents show. If absent, show the six builtins.
|
||||||
|
3. AgentPicker mounts in `ChatInput.tsx` toolbar between ModelPicker and the `+` button. **No `SettingsDrawer.tsx` or `Header.tsx` changes in this batch.**
|
||||||
|
4. SettingsDrawer agent entry + Header active-agent badge moved to Batch 7.
|
||||||
|
|
||||||
|
**Files to create:**
|
||||||
|
|
||||||
|
- `apps/server/src/services/agents.ts` — parser, six builtin defaults, mtime-keyed cache.
|
||||||
|
- `apps/server/src/routes/agents.ts` — `GET /api/projects/:id/agents`.
|
||||||
|
- `apps/web/src/components/AgentPicker.tsx` — dropdown, matches ModelPicker pattern.
|
||||||
|
|
||||||
|
**Files to modify:**
|
||||||
|
|
||||||
|
- `apps/server/src/schema.sql` — `ALTER TABLE sessions ADD COLUMN IF NOT EXISTS agent_id TEXT;`
|
||||||
|
- `apps/server/src/services/inference.ts` — resolution order: `effective_system_prompt`, `effective_model`, `effective_temperature`, `effective_tools` from session + agent + project. Filter tools array against agent whitelist before sending to llama-swap.
|
||||||
|
- `apps/server/src/routes/sessions.ts` — PATCH accepts `agent_id`.
|
||||||
|
- `apps/server/src/types/api.ts` — Agent type, extend Session with `agent_id`.
|
||||||
|
- `apps/web/src/api/client.ts`, `apps/web/src/api/types.ts` — Agent type, `api.agents.list(projectId)`.
|
||||||
|
- `apps/web/src/components/ChatInput.tsx` — mount AgentPicker.
|
||||||
|
|
||||||
|
**Testing plan (manual, before locking temps):**
|
||||||
|
|
||||||
|
- Drop `/opt/boocode/AGENTS.md` (six agents, no `model` field on any).
|
||||||
|
- For each of the 7 keeper models, switch session model and run the same target prompt against each agent. Log tok/s, instruction-following quality.
|
||||||
|
- Adjust per-agent temperature in `AGENTS.md` based on results.
|
||||||
|
- A/B candidates: qwen3.6-35b-a3b-mxfp4 (daily), qwopus3.6-35b-a3b-q4 (reasoning), qwopus3.5-27b-q4, qwen3.6-27b-ud-q4-xl, nemotron-3-nano-30b, gemma-4-26b-a4b-mxfp4, qwen3-coder-30b-apex.
|
||||||
|
|
||||||
|
**Dependencies:** v1.6.2 merged.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.1-batch3 — Pane system ✅
|
### Batch 11 — Architect: codebase map (REVISED)
|
||||||
|
|
||||||
`session_panes` table, pane CRUD with transactional position-shift, Workspace + tab strip + drag-to-reorder (native HTML5), ChatPane (extracted), FileBrowserPane (tree + Shiki + filter), chat→file click, PaneTab context menu, `file_ops` + `file_index` shared services, broker user channel + `/ws/user`, `session_updated`, `session_loaded`, idempotent default-Chat-pane backfill.
|
**Inspiration / lift:** `nmakod/codecontext` (MIT, Go binary).
|
||||||
|
|
||||||
**Schema:** `session_panes` (id, session_id, position, kind CHECK, state JSONB).
|
**What it gives BooCode:** an architect-grade codebase overview without embeddings. Codecontext parses the repo with tree-sitter, extracts symbols, builds import/dependency relationships, and exposes the result via an MCP server with 8 tools. The model gets a structural map of any codebase on demand.
|
||||||
|
|
||||||
|
**Why this replaces the original Batch 11 (aider PageRank port):** codecontext is a finished binary in our stack language (Go), with watch mode, incremental updates, framework detection, and git-co-change-based semantic neighborhoods (no embeddings). The aider port would be reimplementing what codecontext already ships.
|
||||||
|
|
||||||
|
**Scope:**
|
||||||
|
|
||||||
|
- Add `codecontext` sidecar container to `docker-compose.yml`. Mount the project root read-only. One sidecar per BooCode instance — projects are addressed by absolute path.
|
||||||
|
- Wire each codecontext MCP tool into BooCode’s `inference/tools.ts` as a native tool the model can call:
|
||||||
|
- `repo_overview(project_id)` → codecontext `get_codebase_overview`
|
||||||
|
- `repo_file_analysis(project_id, path)` → `get_file_analysis`
|
||||||
|
- `repo_symbol_info(project_id, symbol)` → `get_symbol_info`
|
||||||
|
- `repo_search_symbols(project_id, query)` → `search_symbols`
|
||||||
|
- `repo_dependencies(project_id, path)` → `get_dependencies`
|
||||||
|
- `repo_semantic_neighborhoods(project_id, path)` → `get_semantic_neighborhoods` (git co-change)
|
||||||
|
- `repo_framework_analysis(project_id)` → `get_framework_analysis`
|
||||||
|
- `path_guard.ts` extension: incorporate `continuedev/continue` `DEFAULT_SECURITY_IGNORE_FILETYPES` so codecontext can’t surface `.env`, `.pem`, keys, etc.
|
||||||
|
- Fallback grammars: drop `Aider-AI/aider`‘s `aider/queries/tree-sitter-*.scm` files for any language codecontext doesn’t cover. Use them via an in-process tree-sitter wrapper *only if* a project needs an unsupported language. Defer wrapper build until that’s an actual gap.
|
||||||
|
|
||||||
|
**Where it goes:** new `apps/server/src/architect/` directory. No new tables — codecontext maintains its own state on disk. New env: `CODECONTEXT_URL=http://codecontext:8765` (MCP endpoint).
|
||||||
|
|
||||||
|
**Decisions to make at recon time:**
|
||||||
|
|
||||||
|
- Bundle the binary directly in the BooCode Dockerfile, or run codecontext as its own service? Sidecar is cleaner. Bundle is one less container.
|
||||||
|
- How does the model discover codecontext tools — register them statically in the tools registry, or proxy MCP `tools/list` at startup?
|
||||||
|
|
||||||
|
**Dependencies:** none. Can ship before Batches 5–10.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.1-batch3.5 — Chips + @file + line-select ✅
|
### Batch 11b — Architect: repo health (NEW)
|
||||||
|
|
||||||
`Attachment` type + `flattenToMessage` + LANG_MAP, `AttachmentChip`, `AttachmentPreviewModal`, ChatInput chip-row, hand-rolled `@file` mention popover, line-select-attach in FileBrowserPane via local `FileViewer`, FileBrowserPane filter upgrade (empty=tree, non-empty=flat).
|
**Inspiration / lift:** `spirituslab/codesight` (MIT-ish, TS/Node).
|
||||||
|
|
||||||
-----
|
**What it gives BooCode:** complement to Batch 11. Where codecontext answers “what is this codebase,” repo health answers “what’s wrong with this codebase.” Call graph, circular dependency detection, dead code flagging.
|
||||||
|
|
||||||
### v1.2 — Chats inside sessions ✅
|
**Scope:**
|
||||||
|
|
||||||
Originally planned as the mobile pass. Reshuffled: structural refactor — chats inside sessions, right-rail, `/compact` (chat’s own model summarizes via `kind='compact'` system message), force-send.
|
- Port codesight’s `analyze.mjs` analyzer core into `apps/server/src/architect/repo_health.ts`. Drop the VS Code extension shell. Keep:
|
||||||
|
- Symbol extraction (already overlaps codecontext — call codecontext where possible, only redo what’s needed for graph edges).
|
||||||
**Schema:** `chats` table, `sessions.status`, `messages.chat_id`, `messages.kind` (regular | compact).
|
- Call graph builder (function-to-function edges).
|
||||||
|
- Circular dependency detector.
|
||||||
-----
|
- Dead code detector (exported symbols never imported or called).
|
||||||
|
- New tool: `repo_health(project_id)` returning `{ circular_dependencies: [...], dead_code: [...] }`. Output respects codesight’s documented false-positive caveats (customElements.define, framework entry points, dynamic imports) — surface those in the tool description so the model doesn’t trust dead-code flags blindly.
|
||||||
### v1.2-project-ux ✅
|
- Cache results in `boocode_db` keyed by `(project_id, file_hashes)`. Invalidate on file change via file-index hash check.
|
||||||
|
|
||||||
Full new-project bootstrap (mkdir + git init + .gitignore + first commit + Gitea remote + push), sidebar context menu (Rename / Archive / Open in Gitea), project landing page archived-list, Gitea API integration. Option B taken: `BOOTSTRAP_ROOT` env var, `/opt` stays read-only mount, `/opt/projects` writable.
|
|
||||||
|
|
||||||
**Schema:** `projects.status`, `projects.archived_at`.
|
|
||||||
|
|
||||||
**Key decisions:**
|
|
||||||
|
|
||||||
- `execFile` only, no `exec` shell strings.
|
|
||||||
- DB INSERT last in bootstrap sequence.
|
|
||||||
- Soft-fail on Gitea steps.
|
|
||||||
- Project Delete endpoint exists but stays unexposed (re-add INSERTs fresh row → FK cascade nukes history; archive is the safe pattern).
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.3 — Tab close + chat archive ✅
|
|
||||||
|
|
||||||
Tab close UX cleanup, chat-level archive (separate from session archive).
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.4 — Fork + delete + header polish ✅
|
|
||||||
|
|
||||||
Was originally planned as Batch 5.
|
|
||||||
|
|
||||||
**Shipped:** `POST /api/sessions/:id/fork` (deep copy messages up to target, new session in same project), `DELETE /api/sessions/:id/messages/:id` (cascading via `messages_deleted` frame), header breadcrumb (Projects → Project → Session), inline-editable session name, file path shown when File Browser pane is active, `useActivePane` hook.
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.5 — Refactor + tests + security scoping + context tracker ✅
|
|
||||||
|
|
||||||
5-commit sequence:
|
|
||||||
|
|
||||||
1. **Refactor:** FileBrowserPane (865 → split with FileViewer extracted), Workspace, inference split.
|
|
||||||
1. **Vitest harness:** 23 tests covering routes + resolveProjectPath. Pinned to v3 (Vite 5 / vitest 4 incompatibility).
|
|
||||||
1. **Error-log surfacing:** dead-code removal from earlier H1/H2 audit items, structured error logs to client.
|
|
||||||
1. **Mount scoping:** `/opt:/opt:ro` + `BOOTSTRAP_ROOT` writable subdir. Container loses write to `/opt` proper.
|
|
||||||
1. **Persistent context-window tracker:** floating popover above chat input right edge, source = latest `message_complete` frame’s `ctx_used` / `ctx_max`, color-coded (neutral <60%, amber 60–85%, red 85%+), hides when `ctx_max` null.
|
|
||||||
|
|
||||||
**Carried bug:** `resolveProjectPath` whitelist-root bypass — discovered, asserted as “BEHAVIOR GAP” rather than silently patched. Fix landed in v1.6 (H1).
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.5.1 — Bootstrap hotfix ✅ (`4a9f207`)
|
|
||||||
|
|
||||||
Dockerfile (git installed in container), docker-compose.yml, project_bootstrap.ts (SSH keypair, known_hosts, SSH URL Tailscale rewrite), CreateProjectModal.tsx, .gitignore. /opt/projects label clarified.
|
|
||||||
|
|
||||||
**Known issue carried forward:** dispatch used the in-repo `secrets/boocode_gitea` SSH key because the agent key was rejected. Key exposure flagged. Audit + rotation tracked in v1.6.1 below.
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.6-mobile-pass ✅
|
|
||||||
|
|
||||||
**Merged via 6 commits `57c883b..943ae7d`** (5 functional + 1 docs):
|
|
||||||
|
|
||||||
1. `57c883b chore: fix resolveProjectPath whitelist-root bypass` (H1 — dropped `real !== whitelistReal` short-circuit; flipped the v1.5 BEHAVIOR GAP test; 23/23 pass).
|
|
||||||
1. `a643b5f feat(mobile): viewport hook + sidebar drawer + hamburger headers` (M1 + M2 + M6-header).
|
|
||||||
1. `cd897d6 feat(mobile): single-pane stack + long-press tab menu + swipe-to-close` (M3 + M4 + A2).
|
|
||||||
1. `273eeac feat(mobile): chat input keybinds + safe-area + tap targets + overflow safety` (M5 + M6-bottom + M7 + M8).
|
|
||||||
1. `4b5b9b2 feat(mobile): pull-to-refresh sidebar list` (A1).
|
|
||||||
1. `943ae7d docs: add v1.x roadmap snapshot` (this file).
|
|
||||||
|
|
||||||
**Decisions:**
|
**Decisions:**
|
||||||
|
|
||||||
- H2 (roadmap update) handled in this file rather than by Claude Code.
|
- Build it in-process (Node) vs spawn a CLI? In-process is simpler. Spawn matches codecontext sidecar pattern but adds latency.
|
||||||
- M5: mobile = button-only send, Enter inserts newline. Desktop unchanged. `isComposing` guard for CJK IME.
|
|
||||||
- M6: kept `max-w-[1000px]` (mobile naturally full-width below cap).
|
|
||||||
- URL state: `?pane=<paneId>`. Bare URL resets activePaneIdx to 0.
|
|
||||||
- Long-press dispatches synthetic `contextmenu` on `[data-tab-id]`, opening Radix ContextMenuTrigger at touch coords. iOS callout suppressed.
|
|
||||||
- `SwipeablePaneTab`: 60px threshold, bails if vertical >30px, opacity 1→0.4.
|
|
||||||
- A2 bundled with M3 in Commit 3 (structural coupling).
|
|
||||||
- Home.tsx no hamburger.
|
|
||||||
|
|
||||||
**Deferred from v1.6 → rolled into v1.6.1-cleanup:**
|
**Dependencies:** Batch 11 merged (so we can reuse codecontext’s parse output where possible). Can be deferred until after Batches 5–10.
|
||||||
|
|
||||||
- RightRail still renders on mobile (~32px column).
|
|
||||||
- Secrets hygiene audit.
|
|
||||||
- `ProjectSidebar.tsx` and `ChatTabBar.tsx` share content from two commits each — use `git add -p`.
|
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.6.1-cleanup ✅ (`6a9fe18`)
|
### Batch 12 — Tool approval gating + plan/act mode
|
||||||
|
|
||||||
**Shipped:** RightRail wrapped in `<div className="max-md:hidden contents">` so it's hidden entirely below the md breakpoint on mobile. (Note: v1.6.2 reverses this and replaces with a proper mobile drawer — see below.)
|
**Inspiration / lift:** `cline/cline` (Apache-2.0).
|
||||||
|
|
||||||
**Audited but not shipped (queued for follow-ups):**
|
**What it gives BooCode:** per-session control over which tools the model can call. Lays the groundwork for BooCoder by building the gating mechanism before there are any write tools to gate.
|
||||||
|
|
||||||
- **Secrets hygiene:** `secrets/boocode_gitea` is NOT tracked; never committed to any branch; `.gitignore` already covers `secrets/`. Rotation is a Gitea-side action, no repo change needed.
|
|
||||||
- **`.bak` files:** 3 leftover from v1.5.1 (`docker-compose.yml.bak-20260516`, `Dockerfile.bak-20260516`, `apps/web/src/components/CreateProjectModal.tsx.bak-20260516`). Git-invisible via global `~/.gitignore_global` (`*.bak*`). Decide per file.
|
|
||||||
- **Unused exports:** neither `knip` nor `ts-prune` installed. Proposal pending.
|
|
||||||
- **Dead WS frames:** `session_renamed` HAS a server publisher (`routes/sessions.ts:140`, added in v1.4) — the roadmap's "no server publisher" open item is **STALE**, crossed off. The `InferenceFrame` union still declares `session_renamed` as a type variant but no code publishes it on the per-session channel; trivial 1-line cleanup deferred.
|
|
||||||
- **Unused imports:** web `tsc --noUnusedLocals --noUnusedParameters` returns 0 warnings.
|
|
||||||
- **`useSessionStream` refcount:** opportunity confirmed (~90 lines diff to apply the `useSidebar`-style module-scope singleton pattern). Risk LOW. Queued for v1.6.2 or later.
|
|
||||||
- **PATCH `/api/panes/:id` ownership:** **MOOT** — endpoint does not exist (the pane REST API was never re-introduced after pane state moved to client-side localStorage in v1.2). Crossed off open items.
|
|
||||||
- **Hand-rolled patterns vs library:** 5 hand-rolled hooks/components total 336 lines. None duplicates anything in existing deps; library swap (`@use-gesture`, `react-pull-to-refresh`) not worth the dep cost yet.
|
|
||||||
- **`/opt:/opt:ro` mount tightening:** Two-option plan documented for v1.6.2 — Option A (per-project bind-mounts) or Option B (deny `.env` pattern in `pathGuard`). Option B is the simpler short-term fix.
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.6.2-mobile-ui-fixes 🔄
|
|
||||||
|
|
||||||
**Hand-back received, uncommitted on `v1.6.2-mobile-ui-fixes`.** 4-commit sequence proposed:
|
|
||||||
|
|
||||||
1. `fix(mobile): hide Split button + single-pane navigator chrome` (G1 — wrap the Workspace Split row in `!isMobile`).
|
|
||||||
1. `feat(mobile): rework Session and Project headers for narrow viewports` (G2 — breadcrumb `hidden sm:flex`, session name cap `max-w-[140px] sm:max-w-[280px]`, project page heading `text-base sm:text-lg`, “New session” icon-only on mobile).
|
|
||||||
1. `feat(mobile): add "New chat" to tab long-press context menu` (G3 — top of menu, separator, then existing items).
|
|
||||||
1. `feat(mobile): right-rail as drawer on mobile, header toggle button` (G4 option b — new `useRightRailDrawer` Context hook, `RightRail` renders as fixed `w-[85vw] max-w-sm` drawer on mobile, FolderTree button in Session header, **reverts v1.6.1's `max-md:hidden` wrapper**).
|
|
||||||
|
|
||||||
**Decisions:**
|
|
||||||
|
|
||||||
- G4 option b chosen: mobile file browsing IS useful; drawer pattern mirrors `useSidebarDrawer`.
|
|
||||||
- G2 single-row session-name+model layout (model picker right-aligned), per spec example.
|
|
||||||
- G3 "New chat" at top, separator, then Rename.
|
|
||||||
- G2 "New session" button: icon-only on mobile via `<span className="hidden sm:inline">New session</span>`.
|
|
||||||
|
|
||||||
**Adjacent uncommitted change (not part of v1.6.2):** `MAX_TOOL_LOOP_DEPTH 5 → 15` in `apps/server/src/services/inference.ts`. Sam-authored, sitting in working tree on `v1.6.2-mobile-ui-fixes`. **NOT on main as of this update.** Commit separately.
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.7 — Drag-drop + paste (planned, was Batch 6)
|
|
||||||
|
|
||||||
**Depends on:** v1.6.1 merged.
|
|
||||||
|
|
||||||
**Scope (trimmed — chip infra exists from v1.1-batch3.5):**
|
|
||||||
|
|
||||||
- Drag-drop files onto ChatInput → chip via `addAttachment({kind: 'file', source: 'drop'})`.
|
|
||||||
- Paste >8 lines → chip via `addAttachment({kind: 'paste', source: 'paste'})`. ≤8 lines inline.
|
|
||||||
- Drop overlay (dashed border + “Drop to attach”).
|
|
||||||
- Client-side 5 MB cap + binary detection (null-byte check in first 8KB).
|
|
||||||
- Max 10 attachments shared cap.
|
|
||||||
- Folder drop rejected. Image paste rejected. Binary files rejected with toast.
|
|
||||||
|
|
||||||
**Frontend only.**
|
|
||||||
|
|
||||||
-----
|
|
||||||
|
|
||||||
### v1.8 — Settings drawer (planned, was Batch 7)
|
|
||||||
|
|
||||||
**Depends on:** header gear (already in v1.4).
|
|
||||||
|
|
||||||
**Scope:**
|
**Scope:**
|
||||||
|
|
||||||
- Right-side drawer (hand-rolled, no shadcn Sheet). Tabbed: Session + Project.
|
- New column `sessions.tool_approval_mode TEXT` — values: `read_only` (v1.x default), `plan`, `act_auto`, `act_approve`.
|
||||||
- Session tab: system prompt, web search toggle, model picker, session name.
|
- New column `sessions.approved_tools JSONB` — per-session whitelist for `act_approve` mode.
|
||||||
- Project tab: default system prompt, default web search, project name, root path (read-only), delete project (consider whether to expose given the cascade concern).
|
- Tool registry refactor: tools tagged `read_only` or `write`. In `read_only` mode (v1.x), write tools never appear in the model’s tools array. In `plan` mode, same — write tools hidden, model produces a plan only. `act_*` modes unlock writes (post-v1.x).
|
||||||
- Resolution: `session.system_prompt OR project.default_system_prompt OR ""`.
|
- UI: mode picker in SettingsDrawer (Batch 7 dependency). Inline indicator in chat header.
|
||||||
- Project defaults applied at session create (copied), not retroactively.
|
|
||||||
- Web search toggle persistent per session (`sessions.web_search_enabled`).
|
|
||||||
|
|
||||||
**Schema:** `sessions.web_search_enabled`, `projects.default_system_prompt`, `projects.default_web_search`.
|
**Dependencies:** Batch 7 (SettingsDrawer).
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.9 — Web search backend (planned, was Batch 8)
|
### Batch 13 — Append-only event log
|
||||||
|
|
||||||
**Depends on:** v1.8.
|
**Inspiration / lift:** `OpenHands/OpenHands` V1 (MIT).
|
||||||
|
|
||||||
|
**What it gives BooCode:** replaces the ad-hoc `messages` table semantics with a typed event stream. Unlocks rewind, time-travel, and clean handoff semantics for multi-agent flows.
|
||||||
|
|
||||||
**Scope:**
|
**Scope:**
|
||||||
|
|
||||||
- `web_search` tool → SearXNG at `http://100.114.205.53:8888/search?format=json`, top-N `{title, url, snippet}`.
|
- New `session_events` table: `(id, session_id, ts, kind, payload JSONB, parent_id)`. Event kinds: `user_message`, `assistant_message`, `tool_call`, `tool_result`, `pane_action`, `mode_change`, `system`.
|
||||||
- `web_fetch` tool, regex HTML strip (no cheerio), 50KB cap.
|
- Existing `messages` table becomes a derived view over `session_events` for backward compatibility, then deprecated over a release.
|
||||||
- Tools conditionally included based on `session.web_search_enabled`.
|
- Inference loop emits events instead of mutating message rows.
|
||||||
- `ToolCallCard.tsx` renders results as clickable URL list, web_fetch as text preview.
|
- Frontend `useSessionStream` reducer rewritten to consume events.
|
||||||
- Env: `SEARXNG_URL`, `WEB_FETCH_TIMEOUT_MS`, `WEB_FETCH_MAX_BYTES`.
|
|
||||||
|
**Migration is non-trivial.** Plan in a dedicated batch with explicit cutover window.
|
||||||
|
|
||||||
|
**Dependencies:** Batches 5 (fork/delete) and 7 (settings) merged. Must not be in flight with other backend work.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.10 — Agents (planned, was Batch 9)
|
### Batch 14 — BooCoder: pending changes
|
||||||
|
|
||||||
**Depends on:** v1.8.
|
**Inspiration / lift:** `plandex-ai/plandex` (MIT).
|
||||||
|
|
||||||
|
**What it gives BooCode:** safe write tools. Edits queue in a virtual layer; nothing touches the filesystem until explicit `/apply`.
|
||||||
|
|
||||||
**Scope:**
|
**Scope:**
|
||||||
|
|
||||||
- Tier 2 agents: system prompt + model + temperature + tools whitelist per agent.
|
- New container `boocoder` at `100.114.205.53:9502`. Owns write tools (`edit_file`, `create_file`, `delete_file`, `apply_pending`, `rewind`).
|
||||||
- `AGENTS.md` (OpenCode-compatible): `## Agent Name` blocks with YAML frontmatter.
|
- New table `pending_changes (id, session_id, file_path, diff TEXT, status, created_at)`. Status: `pending`, `applied`, `rejected`.
|
||||||
- Three builtin defaults (Investigator, Architect, Reviewer) when no `AGENTS.md`.
|
- Tools execute against the pending-changes layer, not the filesystem. `apply_pending` is the only path that touches disk. `rewind` rolls back a `pending`-id back to disk state.
|
||||||
- If `AGENTS.md` exists, only its agents shown.
|
- BooCode chat container stays read-only (`/opt:/opt:ro`). BooCoder mounts `/opt/repos:/opt/repos:rw` and uses git worktree pattern from paseo for isolation.
|
||||||
- Agent picker in ChatInput toolbar + SettingsDrawer.
|
- Frontend: new pane kind `pending_diff` shows the queued diff inline with Approve/Reject per chunk.
|
||||||
- Tools whitelist enforced at inference layer. BooChat agents read-only.
|
|
||||||
- File parsed on demand with mtime cache.
|
|
||||||
- Mid-conversation agent switch allowed; old messages retain their tool history.
|
|
||||||
|
|
||||||
**Schema:** `sessions.agent_id TEXT`.
|
**Dependencies:** Batches 12 (gating) + 13 (events). Don’t start until both are live.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
### v1.11 — BooTerm (planned, was Batch 10)
|
### Batch 15 — BooCoder runtime isolation
|
||||||
|
|
||||||
**Depends on:** v1.1-batch3 (pane system), v1.8 (settings drawer pattern).
|
**Inspiration / lift:** `OpenHands/OpenHands` (MIT).
|
||||||
|
|
||||||
|
**What it gives BooCode:** per-session Docker sandbox for BooCoder writes. Closes the `/opt:ro` mount risk identified in v1.x open items.
|
||||||
|
|
||||||
**Scope:**
|
**Scope:**
|
||||||
|
|
||||||
- New container `booterm` at `100.114.205.53:9501`. Fastify + node-pty + tmux.
|
- Per-session container spawned by BooCoder on first write. Container has only the project’s path mounted, not `/opt`.
|
||||||
- Caddy path-based routing: `/api/term/*` + `/ws/term/*` → booterm.
|
- Container lifecycle: spawn on first write call, idle-timeout after 30 min, recreate on resume.
|
||||||
- Shared `boocode_db`.
|
- Action execution server pattern: HTTP API inside the container, BooCoder calls in. Standard OpenHands runtime contract.
|
||||||
- Per-session tmux (`bc-<session_id>`), per-pane tmux window (`term-<pane_id>`).
|
|
||||||
- xterm.js terminal pane. New `kind = 'terminal'` in `session_panes` CHECK.
|
|
||||||
- PTY over binary WebSocket. Resize via `tmux resize-window`.
|
|
||||||
- Workspace mount: `/opt/repos:/opt/repos:rw`. BooCode chat container keeps `/opt:/opt:ro`.
|
|
||||||
- Send-to-terminal from chat: select text → right-click → “Send to terminal”.
|
|
||||||
- tmux persistence across WS reconnects, page refreshes, container restarts.
|
|
||||||
- No chroot/namespace isolation. Acceptable single-user homelab.
|
|
||||||
|
|
||||||
**New app:** `apps/booterm/`.
|
**Dependencies:** Batch 14.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
## Architecture
|
### Batch 16 — Multi-provider LLM abstraction
|
||||||
|
|
||||||
### Containers (current + planned)
|
**Inspiration / lift:** `earendil-works/pi` `pi-ai` (MIT).
|
||||||
|
|
||||||
|Container |Port |Mount |Purpose |Status |
|
**What it gives BooCode:** optional non-llama-swap inference paths (Anthropic, OpenAI, Mistral direct). Currently we have one provider (llama-swap) and the existing `streamCompletion` is hardcoded to OpenAI-compatible at that endpoint.
|
||||||
|------------|---------------------|-----------------------------------|----------------------------|---------|
|
|
||||||
|`boocode` |`100.114.205.53:9500`|`/opt:/opt:ro` + `/opt/projects:rw`|Chat + read-only tools + SPA|Live |
|
|
||||||
|`boocode_db`|`127.0.0.1:5500` |`boocode_pgdata` |Postgres 16-alpine (shared) |Live |
|
|
||||||
|`booterm` |`100.114.205.53:9501`|`/opt/repos:/opt/repos:rw` |Terminal sessions |v1.11 |
|
|
||||||
|`boocoder` |TBD |`/opt/repos:/opt/repos:rw` |Write tools |Post-v1.x|
|
|
||||||
|
|
||||||
### URL routing (target state after v1.11)
|
**Scope:**
|
||||||
|
|
||||||
```
|
- Provider abstraction: `interface LLMProvider { stream(req): AsyncIterator<Frame> }`.
|
||||||
code.indifferentketchup.com
|
- Built-in: llama-swap (current), Anthropic, OpenAI (Codex-style).
|
||||||
├── / → boocode (SPA)
|
- Per-session `provider_id` column.
|
||||||
├── /api/chat/*, /ws/chat/* → boocode :9500
|
|
||||||
├── /api/term/*, /ws/term/* → booterm :9501
|
|
||||||
├── /api/coder/*, /ws/coder/* → boocoder (future)
|
|
||||||
└── /ws/user → boocode :9500
|
|
||||||
```
|
|
||||||
|
|
||||||
### Database
|
**Status:** **Optional. Skip unless a concrete need surfaces.** llama-swap covers daily driver work.
|
||||||
|
|
||||||
Single Postgres `boocode_db`. All containers share. Projects shared. Sessions container-specific.
|
|
||||||
|
|
||||||
Current schema (post v1.5.1):
|
|
||||||
|
|
||||||
```
|
|
||||||
projects
|
|
||||||
├── id UUID PK
|
|
||||||
├── name TEXT
|
|
||||||
├── root_path TEXT
|
|
||||||
├── status TEXT (v1.2-project-ux: active | archived)
|
|
||||||
├── archived_at TIMESTAMPTZ (v1.2-project-ux)
|
|
||||||
├── default_system_prompt TEXT (v1.8)
|
|
||||||
├── default_web_search BOOLEAN (v1.8)
|
|
||||||
└── created_at TIMESTAMPTZ
|
|
||||||
|
|
||||||
sessions
|
|
||||||
├── id UUID PK
|
|
||||||
├── project_id UUID FK → projects
|
|
||||||
├── name TEXT
|
|
||||||
├── model TEXT
|
|
||||||
├── system_prompt TEXT
|
|
||||||
├── status TEXT (v1.2: active | archived)
|
|
||||||
├── web_search_enabled BOOLEAN (v1.8)
|
|
||||||
├── agent_id TEXT (v1.10)
|
|
||||||
├── created_at TIMESTAMPTZ
|
|
||||||
└── updated_at TIMESTAMPTZ
|
|
||||||
|
|
||||||
chats (v1.2)
|
|
||||||
├── id UUID PK
|
|
||||||
├── session_id UUID FK → sessions
|
|
||||||
├── name TEXT
|
|
||||||
├── status TEXT
|
|
||||||
├── created_at TIMESTAMPTZ
|
|
||||||
└── updated_at TIMESTAMPTZ
|
|
||||||
|
|
||||||
messages
|
|
||||||
├── id UUID PK
|
|
||||||
├── session_id UUID FK → sessions
|
|
||||||
├── chat_id UUID FK → chats (v1.2)
|
|
||||||
├── kind TEXT (v1.2: regular | compact)
|
|
||||||
├── role TEXT
|
|
||||||
├── content TEXT
|
|
||||||
├── tool_calls JSONB
|
|
||||||
├── tool_results JSONB
|
|
||||||
├── status TEXT
|
|
||||||
├── last_seq INTEGER
|
|
||||||
├── tokens_used INTEGER (v1.1-batch1)
|
|
||||||
├── ctx_used INTEGER (v1.1-batch1)
|
|
||||||
├── ctx_max INTEGER (v1.1-batch1)
|
|
||||||
├── started_at TIMESTAMPTZ (v1.1-batch1)
|
|
||||||
├── finished_at TIMESTAMPTZ (v1.1-batch1)
|
|
||||||
└── created_at TIMESTAMPTZ
|
|
||||||
|
|
||||||
session_panes (v1.1-batch3)
|
|
||||||
├── id UUID PK
|
|
||||||
├── session_id UUID FK → sessions (CASCADE)
|
|
||||||
├── position INTEGER
|
|
||||||
├── kind TEXT CHECK (chat | file_browser | terminal)
|
|
||||||
├── state JSONB
|
|
||||||
└── created_at TIMESTAMPTZ
|
|
||||||
|
|
||||||
settings
|
|
||||||
├── k TEXT PK
|
|
||||||
└── v TEXT
|
|
||||||
```
|
|
||||||
|
|
||||||
### Reusable patterns
|
|
||||||
|
|
||||||
|Pattern |Where |Used by |
|
|
||||||
|----------------------------|----------------------------------------|---------------------------------------------------------|
|
|
||||||
|In-app event bus |`sessionEvents.ts` |All batches. Module-scope `Set<Listener>`. |
|
|
||||||
|Singleton hooks |`useSidebar.ts` |Module-scope shared state. |
|
|
||||||
|User-channel WS broker |`broker.ts` + `useUserEvents.ts` |Cross-tab lifecycle. One WS per tab. |
|
|
||||||
|`clock_timestamp()` |All INSERT/UPDATE |Never `NOW()` in new code. |
|
|
||||||
|Additive schema only |`schema.sql` |`ADD COLUMN IF NOT EXISTS`, `CREATE TABLE IF NOT EXISTS`.|
|
|
||||||
|Idempotent backfills |`schema.sql` |`INSERT ... WHERE NOT EXISTS`. |
|
|
||||||
|`enable_thinking: false` |`auto_name.ts` |Required for Qwen3 utility calls. |
|
|
||||||
|`pathGuard` |`tools/*`, `file_ops.ts` |Realpath + project root enforcement. |
|
|
||||||
|Shared `file_ops.ts` |`tools.ts`, `routes/projects.ts` |Same core for inference tools and UI. |
|
|
||||||
|File index (`file_index.ts`)|`routes/projects.ts` |`rg --files` + mtime cache. |
|
|
||||||
|`useViewport` |`hooks/useViewport.ts` (v1.6) |matchMedia, SSR-safe. |
|
|
||||||
|`useSidebarDrawer` |`hooks/useSidebarDrawer.tsx` (v1.6) |Context + auto-close on route change. |
|
|
||||||
|Hand-rolled long-press |`hooks/useLongPress.ts` (v1.6) |500ms touchstart timer, dispatches synthetic contextmenu.|
|
|
||||||
|Hand-rolled pull-to-refresh |`hooks/usePullToRefresh.ts` (v1.6) |80px threshold, 600ms min hold. |
|
|
||||||
|Hand-rolled swipe |`components/SwipeablePaneTab.tsx` (v1.6)|60px threshold, vertical bail at 30px. |
|
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
## Tech stack
|
### Batch 17 — Workflow graphs
|
||||||
|
|
||||||
|Layer |Tech |
|
**Inspiration / lift:** `microsoft/agent-framework` (MIT) — concepts only.
|
||||||
|----------------|--------------------------------------------------------------------------|
|
|
||||||
|Backend |Node 20 + Fastify + `@fastify/websocket` + `@fastify/static` + zod + `pg` |
|
**What it gives BooCode:** multi-agent coordination. Architect → Coder → Reviewer → Verifier handoffs orchestrated by a YAML-defined workflow.
|
||||||
|Frontend |React + Vite + Tailwind v4 + shadcn nova preset |
|
|
||||||
|Inference |llama-swap `http://100.101.41.16:8401` (OpenAI-compatible) |
|
**Status:** **Far future.** Read agent-framework’s `docs/decisions/` ADRs. Don’t port code — Azure/.NET-heavy.
|
||||||
|Search |SearXNG `http://100.114.205.53:8888` (v1.9) |
|
|
||||||
|Syntax |Shiki (`github-dark`) |
|
**Dependencies:** Batches 12 (modes), 13 (events). Realistically a v2.x topic.
|
||||||
|Terminal |xterm.js + node-pty + tmux (v1.11) |
|
|
||||||
|Auth |`Remote-User` from Authelia via Caddy `forward_auth` |
|
|
||||||
|Containerization|Docker Compose, Node 20-alpine, multi-stage, ripgrep apk, git apk (v1.5.1)|
|
|
||||||
|DB |Postgres 16-alpine, loopback `127.0.0.1:5500` |
|
|
||||||
|Networking |Tailscale mesh, Caddy (DO droplet), Authelia SSO |
|
|
||||||
|Code hosting |Gitea `git.indifferentketchup.com` |
|
|
||||||
|Tests |vitest v3 (pinned, Vite 5 / vitest 4 incompatible) |
|
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
## Known open items
|
## Order of operations
|
||||||
|
|
||||||
- **`useSessionStream` refcount.** Two ChatPanes = two WS. Apply singleton pattern. Audited in v1.6.1, queued.
|
Two tracks. Pick one to drive next.
|
||||||
- **`/opt:/opt:ro` mount exposes all `.env` files.** Whitelist scope before BooCoder. Two-option plan documented in v1.6.1 audit; ship in v1.6.2 or v1.7.
|
|
||||||
- **`secrets/boocode_gitea` in repo working tree.** Never committed (git-invisible via global ignore). Rotate the Gitea-side key when convenient; no repo action required.
|
|
||||||
- **Dormant in-boolab BooCode mode.** Reference only.
|
|
||||||
- **BooCoder container.** Post-v1.x.
|
|
||||||
|
|
||||||
**Closed since last update:**
|
**Track A — Finish v1.x mobile + polish then agents:**
|
||||||
|
|
||||||
- ~~`session_renamed` no server WS publisher~~ — server publishes via `broker.publishUser` from `routes/sessions.ts:140` (added in v1.4). Confirmed in v1.6.1 audit.
|
- v1.6.2 ships (in flight)
|
||||||
- ~~PATCH `/api/panes/:id` lacks session-ownership check~~ — endpoint does not exist; the pane REST API was never re-introduced after v1.2 moved pane state to localStorage.
|
- **Batch 9 (agents)** — decoupled, can land next; no UI dependency on 5/6/7
|
||||||
|
- Batches 5, 6, 7, 8 in order. Each is small, frontend-heavy, no architecture risk. Batch 7 absorbs SettingsDrawer agent entry.
|
||||||
|
|
||||||
|
**Track B — Begin architect capabilities in parallel:**
|
||||||
|
|
||||||
|
- Batch 11 (codecontext sidecar) — biggest single capability jump. Frontend stays the same; new tools appear to the model.
|
||||||
|
- Batch 11b (repo health) — follow-up.
|
||||||
|
- Batch 12 (gating) — sets up everything post-v1.x.
|
||||||
|
|
||||||
|
Recommendation: ship v1.6.2, then **Batch 9 (agents)** next so the test bed exists before Track A continues. Then Track A through Batch 7. Batch 11 can run in parallel with Batches 8–10 since 11 has no UI dependency.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
## Dependency graph
|
## Architecture target state
|
||||||
|
|
||||||
```
|
### Containers
|
||||||
v1.0 (initial)
|
|
||||||
│
|
|Container |Port |Mount |Purpose |Status |
|
||||||
▼
|
|-------------|--------------------------------|--------------------------|------------------------------|--------|
|
||||||
v1.1-batch1 (markdown)
|
|`boocode` |`100.114.205.53:9500` |`/opt:/opt:ro` |Chat + read-only tools + SPA |Live |
|
||||||
│
|
|`boocode_db` |`127.0.0.1:5500` |`boocode_pgdata` volume |Postgres 16-alpine |Live |
|
||||||
▼
|
|`codecontext`|`100.114.205.53:8765` (internal)|project root :ro |MCP server for architect tools|Batch 11|
|
||||||
v1.1-batch2 (sidebar)
|
|`booterm` |`100.114.205.53:9501` |`/opt/repos:/opt/repos:rw`|Terminals (tmux + node-pty) |Batch 10|
|
||||||
│
|
|`boocoder` |`100.114.205.53:9502` |per-session sandbox |Write tools |Batch 14|
|
||||||
▼
|
|
||||||
v1.1-batch3 (panes) ────────────────────────┐
|
### Schema additions
|
||||||
│ │
|
|
||||||
▼ │
|
**Batch 9:** `sessions.agent_id TEXT` (nullable; references AGENTS.md by slug).
|
||||||
v1.1-batch3.5 (chips) ──────┐ │
|
**Batch 11:** none (codecontext stateless on disk).
|
||||||
│ │ │
|
**Batch 11b:** `repo_health_cache (project_id, file_hashes_sig, payload JSONB, created_at)`.
|
||||||
▼ │ │
|
**Batch 12:** `sessions.tool_approval_mode`, `sessions.approved_tools`.
|
||||||
v1.2 (chats-in-sessions) │ │
|
**Batch 13:** `session_events`; deprecate `messages` long-tail.
|
||||||
│ │ │
|
**Batch 14:** `pending_changes`.
|
||||||
▼ │ │
|
|
||||||
v1.2-project-ux │ │
|
-----
|
||||||
│ │ │
|
|
||||||
▼ │ │
|
## Lift sources (summary)
|
||||||
v1.3 (tab-close) │ │
|
|
||||||
│ │ │
|
Full inventory in `boocode_code_review.md`. Headline items:
|
||||||
▼ │ │
|
|
||||||
v1.4 (fork+delete+header) ◄──┼────────────────┘
|
|Source |Used for |Where |
|
||||||
│ │
|
|--------------------------------------|----------------------------------------|---------------------|
|
||||||
▼ │
|
|nmakod/codecontext (MIT, Go) |Architect: codebase map sidecar |Batch 11 |
|
||||||
v1.5 (refactor+tests+ctx) │
|
|spirituslab/codesight (MIT-ish, TS) |Architect: repo health analyzer |Batch 11b |
|
||||||
│ │
|
|Aider-AI/aider (Apache-2.0) |Fallback `.scm` grammars (60+ languages)|Batch 11 (fallback) |
|
||||||
▼ │
|
|continuedev/continue (Apache-2.0) |DEFAULT_SECURITY_IGNORE_FILETYPES |Batch 11 prep |
|
||||||
v1.5.1 (bootstrap hotfix) │
|
|cline/cline (Apache-2.0) |Plan/Act mode pattern |Batch 12 |
|
||||||
│ │
|
|plandex-ai/plandex (MIT) |Pending-changes data model |Batch 14 |
|
||||||
▼ │
|
|OpenHands/OpenHands (MIT) |Event log + sandbox runtime |Batches 13, 15 |
|
||||||
v1.6-mobile-pass │
|
|aimasteracc/tree-sitter-analyzer (MIT)|Outline-first response patterns |Reference |
|
||||||
│ │
|
|earendil-works/pi (MIT) |Multi-provider LLM |Batch 16 (optional) |
|
||||||
▼ │
|
|rshah515/claude-code-subagents (MIT) |Reference for builtin agent prompts |Batch 9 (six builtins)|
|
||||||
v1.6.1-cleanup │
|
|microsoft/agent-framework (MIT) |Workflow concepts |Batch 17 (far future)|
|
||||||
│ │
|
|
||||||
▼ │
|
-----
|
||||||
v1.6.2-mobile-ui-fixes ◄─────┘
|
|
||||||
│
|
## Decisions log
|
||||||
▼
|
|
||||||
v1.7 (drag-drop) ◄── v1.1-batch3.5
|
- **Embeddings dropped from BooCode.** Replaced RAG with file-view tools + sidecar analyzers.
|
||||||
│
|
- **Original Batch 11 (aider PageRank port) replaced** by codecontext sidecar approach.
|
||||||
▼
|
- **Original Batch 12 (codebase indexer w/ Harrier) removed** entirely. No embedding infrastructure in BooCode v1.x.
|
||||||
v1.8 (settings)
|
- **Globstar parked** — not an architect tool, future verify-before-commit candidate only.
|
||||||
│
|
- **codeprysm rejected** — embedding-based; node/edge taxonomy noted as reference if we ever build our own graph.
|
||||||
├──▶ v1.9 (web search)
|
- **Batch 9 decoupled from Batch 7 (2026-05-16).** AgentPicker mounts in `ChatInput.tsx` toolbar only. SettingsDrawer agent entry and Header active-agent badge moved to Batch 7. Builtin defaults shipped: six agents (Code Reviewer, Debugger, Refactorer, Architect, Security Auditor, Prompt Builder) with no `model` field — session model wins by default.
|
||||||
│
|
|
||||||
├──▶ v1.10 (agents)
|
|
||||||
│
|
|
||||||
└──▶ v1.11 (BooTerm) ◄── v1.1-batch3
|
|
||||||
```
|
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
## Workflow
|
## Workflow
|
||||||
|
|
||||||
1. Verify previous version merged to `main`.
|
Each batch:
|
||||||
1. Dispatch prompt via Paseo (Claude Code runs at `/opt/boocode`).
|
|
||||||
1. Claude Code recon → blocking questions → implement → hand back.
|
1. Verify previous batch merged.
|
||||||
1. Review hand-back in separate Claude chat (spec compliance, code quality, drift, stale code).
|
2. Dispatch via Paseo to Claude Code at `/opt/boocode`.
|
||||||
1. Deploy: `docker compose up --build -d`.
|
3. Claude Code recon → blocking questions → implement → hand back.
|
||||||
1. Smoke test per the hand-back’s plan.
|
4. Compliance review in separate Claude chat.
|
||||||
1. Sam commits manually, pushes to Gitea, merges to `main`.
|
5. Deploy: `docker compose up --build -d`.
|
||||||
1. Next version.
|
6. Smoke test.
|
||||||
|
7. Sam commits and pushes.
|
||||||
|
|
||||||
Sam reviews all diffs. Sam commits. Never git pull/push/commit on his behalf.
|
Sam reviews all diffs. Sam commits. Never git pull/push/commit on his behalf.
|
||||||
|
|||||||
Reference in New Issue
Block a user