chore: update .gitignore to include office-map.png and remove optimistic message insertion in OfficeProvider

Author: fwartner

.env.example

@@ -1,10 +1,10 @@
 # Agent Office configuration
 
-# Database (leave empty for SQLite default — zero config)
+# Database (leave empty for SQLite default - zero config)
 # For PostgreSQL: DATABASE_URL=postgres://user:pass@localhost:5432/agent_office
 # DATABASE_URL=
 
-# Telegram Bot (optional — get token from @BotFather)
+# Telegram Bot (optional - get token from @BotFather)
 # TELEGRAM_BOT_TOKEN=
 
 # Integrations (all optional)

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,6 +1,6 @@
 ## Summary
 
-<!-- What does this PR do? Keep it brief — 1-3 bullet points. -->
+<!-- What does this PR do? Keep it brief - 1-3 bullet points. -->
 
 ## Changes
 

.gitignore

@@ -23,7 +23,7 @@ Thumbs.db
 *.log
 npm-debug.log*
 
-# Paid assets — must be purchased separately
+# Paid assets - must be purchased separately
 assets/pixelart/Office Tileset/
 assets/pixelart/office-map.png
 

CLAUDE.md

@@ -4,7 +4,7 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 ## What This Is
 
-Agent Office is a pixel-art virtual office for AI agent teams. Agents have live presence states, sit in themed rooms, and can be managed through a retro-styled UI or Telegram bot. Early development — APIs and features may change without notice.
+Agent Office is a pixel-art virtual office for AI agent teams. Agents have live presence states, sit in themed rooms, and can be managed through a retro-styled UI or Telegram bot. Early development - APIs and features may change without notice.
 
 ## Commands
 
@@ -34,15 +34,15 @@ Run a single test file: `npx vitest run src/__tests__/app.test.tsx`
 
 **Data flow:** `OfficeProvider` (React context) polls `GET /api/office/snapshot` every 2-3 seconds. Mutations go through provider methods → POST/PATCH/DELETE endpoints → DB or state file → next poll picks up changes.
 
-**Event system:** `src/server/events.ts` — EventEmitter-based pub/sub. API routes emit events, adapters (webhook, Telegram, Slack, GitHub, Linear) subscribe.
+**Event system:** `src/server/events.ts` - EventEmitter-based pub/sub. API routes emit events, adapters (webhook, Telegram, Slack, GitHub, Linear) subscribe.
 
 ### Key Files
 
 | File | Role |
 |------|------|
-| `src/App.tsx` | Main UI — map canvas, room overlays, agent sprites, sidebar panels, forms |
-| `src/office-provider.tsx` | React context — polling, state management, CRUD wrappers, validation |
-| `src/data.ts` | Seed data — agents, rooms, seats, workday policy |
+| `src/App.tsx` | Main UI - map canvas, room overlays, agent sprites, sidebar panels, forms |
+| `src/office-provider.tsx` | React context - polling, state management, CRUD wrappers, validation |
+| `src/data.ts` | Seed data - agents, rooms, seats, workday policy |
 | `src/world.ts` | Sprite definitions, animation metadata, character sets |
 | `src/office-state.ts` | TypeScript type definitions |
 | `src/server/api-routes.ts` | Shared API route handlers (pure functions) |
@@ -71,26 +71,26 @@ Run a single test file: `npx vitest run src/__tests__/app.test.tsx`
 
 Shared route handlers in `src/server/api-routes.ts`, used by both dev and prod:
 
-- `GET /api/health` — health check (status, version, DB type, uptime)
-- `GET /api/office/snapshot` — full state
-- `POST /api/office/agent` — create agent
-- `PUT /api/office/agent/:id` — full update
-- `PATCH /api/office/agent/:id` — partial update (whitelisted fields only)
-- `DELETE /api/office/agent/:id` — remove agent + cascade
-- `POST /api/office/assign` — queue task
-- `PATCH /api/office/assignment/:id` — update assignment status
-- `GET /api/office/assignments` — query assignments with filters
-- `POST /api/office/activity` — log activity
-- `POST /api/office/decision` — create decision
-- `PATCH /api/office/decision/:id` — update decision
-- `POST /api/office/message` — send message
-- `GET /api/office/messages` — query messages
-- `POST /api/office/room` — create room
-- `PUT /api/office/room/:id` — update room
-- `DELETE /api/office/room/:id` — delete room
-- `POST /api/office/webhook` — create webhook
-- `DELETE /api/office/webhook/:id` — delete webhook
-- `PATCH /api/office/settings` — update settings
+- `GET /api/health` - health check (status, version, DB type, uptime)
+- `GET /api/office/snapshot` - full state
+- `POST /api/office/agent` - create agent
+- `PUT /api/office/agent/:id` - full update
+- `PATCH /api/office/agent/:id` - partial update (whitelisted fields only)
+- `DELETE /api/office/agent/:id` - remove agent + cascade
+- `POST /api/office/assign` - queue task
+- `PATCH /api/office/assignment/:id` - update assignment status
+- `GET /api/office/assignments` - query assignments with filters
+- `POST /api/office/activity` - log activity
+- `POST /api/office/decision` - create decision
+- `PATCH /api/office/decision/:id` - update decision
+- `POST /api/office/message` - send message
+- `GET /api/office/messages` - query messages
+- `POST /api/office/room` - create room
+- `PUT /api/office/room/:id` - update room
+- `DELETE /api/office/room/:id` - delete room
+- `POST /api/office/webhook` - create webhook
+- `DELETE /api/office/webhook/:id` - delete webhook
+- `PATCH /api/office/settings` - update settings
 
 ### Validation Constants (in src/server/validation.ts)
 
@@ -102,19 +102,19 @@ Shared route handlers in `src/server/api-routes.ts`, used by both dev and prod:
 
 ### TypeScript Configuration
 
-- `tsconfig.json` — Frontend (React/Vite), excludes src/db, src/server, src/bot
-- `tsconfig.server.json` — Server modules (Node.js), includes src/db, src/server, src/bot
+- `tsconfig.json` - Frontend (React/Vite), excludes src/db, src/server, src/bot
+- `tsconfig.server.json` - Server modules (Node.js), includes src/db, src/server, src/bot
 
 ## Testing
 
 Tests live in `src/__tests__/` (component/unit) and `tests/` (API integration). 167+ tests covering data integrity, sprites, error boundary, provider logic, UI rendering, agent CRUD, accessibility, and API validation.
 
 ## Code Conventions
 
-- TypeScript strict mode — no `any` types
+- TypeScript strict mode - no `any` types
 - Functional React with hooks only (except ErrorBoundary)
-- All CSS in `src/styles.css` — no CSS-in-JS, uses CSS custom properties
-- Pixel-art aesthetic with Press Start 2P font — keep retro theme consistent
+- All CSS in `src/styles.css` - no CSS-in-JS, uses CSS custom properties
+- Pixel-art aesthetic with Press Start 2P font - keep retro theme consistent
 - WCAG 2.1 AA accessibility: ARIA landmarks, keyboard navigation, focus indicators
 - Commit messages: conventional style (`fix:`, `feat:`, `test:`, etc.)
-- No separate linter configured — TypeScript strict mode is the quality gate
+- No separate linter configured - TypeScript strict mode is the quality gate

CONTRIBUTING.md

@@ -24,10 +24,10 @@ To re-run setup: `npm run setup:force`
 
 ## Code Style
 
-- TypeScript strict mode — no `any` types
+- TypeScript strict mode - no `any` types
 - Functional React components with hooks
-- CSS in `src/styles.css` — no CSS-in-JS
-- Pixel-art aesthetic — keep the retro theme consistent
+- CSS in `src/styles.css` - no CSS-in-JS
+- Pixel-art aesthetic - keep the retro theme consistent
 
 ## Testing
 
@@ -49,7 +49,7 @@ Use concise, descriptive messages:
 
 ## Pull Requests
 
-- Keep PRs focused — one feature or fix per PR
+- Keep PRs focused - one feature or fix per PR
 - Fill in the PR template
 - Ensure CI passes before requesting review
 - Include screenshots for UI changes
@@ -64,5 +64,5 @@ Use the [feature request template](https://github.com/fwartner/agent-office/issu
 
 ## Maintainer
 
-**Florian Wartner** — [Pixel & Process](https://pixelandprocess.de)
+**Florian Wartner** - [Pixel & Process](https://pixelandprocess.de)
 Email: [florian@wartner.io](mailto:florian@wartner.io)

README.md

@@ -1,9 +1,9 @@
 # Agent Office
 
-A pixel-art virtual office where your AI agents live, work, and talk to each other. Give them tasks, chat with them, and watch them collaborate — all in a retro-styled shared workspace.
+A pixel-art virtual office where your AI agents live, work, and talk to each other. Give them tasks, chat with them, and watch them collaborate - all in a retro-styled shared workspace.
 
 > [!WARNING]
-> **Early Development** — This project is under active development. Things will break, features are incomplete, and APIs may change without notice. If something doesn't work, it probably just hasn't been built yet. Contributions and bug reports are welcome!
+> **Early Development** - This project is under active development. Things will break, features are incomplete, and APIs may change without notice. If something doesn't work, it probably just hasn't been built yet. Contributions and bug reports are welcome!
 
 [![CI](https://github.com/fwartner/agent-office/actions/workflows/ci.yml/badge.svg)](https://github.com/fwartner/agent-office/actions/workflows/ci.yml)
 [![Docker](https://github.com/fwartner/agent-office/actions/workflows/docker.yml/badge.svg)](https://github.com/fwartner/agent-office/actions/workflows/docker.yml)
@@ -15,19 +15,19 @@ A pixel-art virtual office where your AI agents live, work, and talk to each oth
 
 ## Features
 
-- **Pixel-art office** — agents as animated sprites on a tiled map with themed rooms
-- **Live presence** — see who's active, paused, blocked, or off hours in real time
-- **Chat with agents** — message one agent or broadcast to all; they respond autonomously via Claude Code
-- **Speech bubbles** — agents show what they're working on right on the map
-- **Task assignment** — queue tasks with priority and watch agents work through them
-- **Auto-saved results** — every task result saved to disk automatically
-- **Markdown everywhere** — chat, tasks, results, and activity all rendered as markdown
-- **Telegram bot** — manage agents and chat from Telegram; messages sync both ways
-- **Integrations** — Slack, GitHub, and Linear notifications out of the box
-- **Configurable office hours** — set timezone, working days, and hours
-- **Decisions & voting** — propose and vote on decisions as a team
-- **Accessible** — WCAG 2.1 AA compliant with keyboard navigation and screen reader support
-- **Mobile friendly** — works on phones and tablets
+- **Pixel-art office** - agents as animated sprites on a tiled map with themed rooms
+- **Live presence** - see who's active, paused, blocked, or off hours in real time
+- **Chat with agents** - message one agent or broadcast to all; they respond autonomously via Claude Code
+- **Speech bubbles** - agents show what they're working on right on the map
+- **Task assignment** - queue tasks with priority and watch agents work through them
+- **Auto-saved results** - every task result saved to disk automatically
+- **Markdown everywhere** - chat, tasks, results, and activity all rendered as markdown
+- **Telegram bot** - manage agents and chat from Telegram; messages sync both ways
+- **Integrations** - Slack, GitHub, and Linear notifications out of the box
+- **Configurable office hours** - set timezone, working days, and hours
+- **Decisions & voting** - propose and vote on decisions as a team
+- **Accessible** - WCAG 2.1 AA compliant with keyboard navigation and screen reader support
+- **Mobile friendly** - works on phones and tablets
 
 ## Getting Started
 
@@ -49,7 +49,7 @@ The map background uses the [Office Tileset by Donarg](https://donarg.itch.io/of
 cp "assets/pixelart/Office Tileset/Office Designs/Office Level 4.png" assets/pixelart/office-map.png
 ```
 
-The app works without it — you'll just see a grid background instead.
+The app works without it - you'll just see a grid background instead.
 
 ### 3. Run
 
@@ -115,4 +115,4 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for development setup and guidelines.
 
 ## Maintainer
 
-**Florian Wartner** — [Pixel & Process](https://pixelandprocess.de)
+**Florian Wartner** - [Pixel & Process](https://pixelandprocess.de)

agent-runtime.mjs

@@ -1,5 +1,5 @@
 /**
- * Agent Runtime — Process manager for Claude Code agent spawning.
+ * Agent Runtime - Process manager for Claude Code agent spawning.
  * Shared by both vite.config.ts (dev) and server.mjs (prod).
  */
 import { spawn } from 'node:child_process'
@@ -71,11 +71,11 @@ export function unregisterAgent(agentId) {
 export function dispatchTask(agentId, assignment, callbacks) {
   const entry = registry.get(agentId)
   if (!entry) {
-    console.warn(`[agent-runtime] Cannot dispatch — agent ${agentId} not registered`)
+    console.warn(`[agent-runtime] Cannot dispatch - agent ${agentId} not registered`)
     return false
   }
   if (entry.currentTask) {
-    console.warn(`[agent-runtime] Cannot dispatch — agent ${agentId} is busy with ${entry.currentTask.assignmentId}`)
+    console.warn(`[agent-runtime] Cannot dispatch - agent ${agentId} is busy with ${entry.currentTask.assignmentId}`)
     return false
   }
 

package.json

@@ -1,7 +1,7 @@
 {
   "name": "agent-office",
   "version": "0.3.0",
-  "description": "Pixel-art virtual office for AI agent teams — live presence, room navigation, agent CRUD, and task assignment in a retro-styled shared workspace.",
+  "description": "Pixel-art virtual office for AI agent teams - live presence, room navigation, agent CRUD, and task assignment in a retro-styled shared workspace.",
   "type": "module",
   "license": "MIT",
   "author": "Florian Wartner <florian@wartner.io> (https://pixelandprocess.de)",

server.mjs

@@ -128,7 +128,7 @@ apiCtx.onSettingsChanged = (settings) => {
 }
 
 // ── HTTP helpers ─────────────────────────────────────
-/** Returns { parsed, raw } — parsed is the JSON object, raw is the string for signature verification */
+/** Returns { parsed, raw } - parsed is the JSON object, raw is the string for signature verification */
 function readBody(req) {
   return new Promise((resolve, reject) => {
     let body = ''
@@ -173,7 +173,7 @@ const server = http.createServer(async (req, res) => {
     return
   }
 
-  // API routes — delegate to shared layer
+  // API routes - delegate to shared layer
   if (url.pathname.startsWith('/api/')) {
     try {
       let parsed = null
@@ -212,7 +212,7 @@ const server = http.createServer(async (req, res) => {
     }
   }
 
-  // Static files — assets
+  // Static files - assets
   let filePath = path.join(__dirname, url.pathname)
   if (url.pathname.startsWith('/assets/') && isSafePath(filePath) && fs.existsSync(filePath)) {
     const ext = path.extname(filePath)
@@ -221,7 +221,7 @@ const server = http.createServer(async (req, res) => {
     return
   }
 
-  // Static files — dist
+  // Static files - dist
   filePath = path.join(DIST, url.pathname === '/' ? 'index.html' : url.pathname)
   if (isSafePath(filePath) && fs.existsSync(filePath) && fs.statSync(filePath).isFile()) {
     const ext = path.extname(filePath)
@@ -300,7 +300,7 @@ server.listen(PORT, '0.0.0.0', async () => {
 
 // ── Graceful shutdown ────────────────────────────────
 function shutdown(signal) {
-  console.log(`\n${signal} received — shutting down gracefully`)
+  console.log(`\n${signal} received - shutting down gracefully`)
   shutdownSSE()
   shutdownAll()
   stopBot()

setup.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 /**
- * Agent Office — First-run setup wizard.
+ * Agent Office - First-run setup wizard.
  *
  * Walks through database backend, Telegram bot, integrations, and port.
  * Writes .env file and initialises seed state.
@@ -62,7 +62,7 @@ function select(question, options, defaultIdx = 0) {
     console.log(`  ${question}`)
     options.forEach((opt, i) => {
       const marker = i === defaultIdx ? `${c.cyan}>${c.reset}` : ' '
-      console.log(`  ${marker} ${c.bold}${i + 1}${c.reset}) ${opt.label}${opt.desc ? ` ${c.dim}— ${opt.desc}${c.reset}` : ''}`)
+      console.log(`  ${marker} ${c.bold}${i + 1}${c.reset}) ${opt.label}${opt.desc ? ` ${c.dim}- ${opt.desc}${c.reset}` : ''}`)
     })
     rl.question(`  ${c.dim}Choice [${defaultIdx + 1}]:${c.reset} `, answer => {
       const idx = parseInt(answer.trim()) - 1
@@ -181,14 +181,14 @@ async function main() {
       envVars.TELEGRAM_BOT_TOKEN = token
       console.log(`  ${c.green}✓${c.reset} Telegram bot configured`)
     } else {
-      console.log(`  ${c.yellow}!${c.reset} No token provided — bot disabled`)
+      console.log(`  ${c.yellow}!${c.reset} No token provided - bot disabled`)
     }
   }
 
   // ── Step 3: Integrations ──────────────────────────
   step('Integrations (optional)')
 
-  console.log(`  ${c.dim}Connect to external services. All optional — skip to use later.${c.reset}`)
+  console.log(`  ${c.dim}Connect to external services. All optional - skip to use later.${c.reset}`)
 
   const useSlack = await confirm('Enable Slack notifications?', false)
   if (useSlack) {