agent/pi - pi - Penguin

Expose agent, agentInterface, and artifactsPanel as public in ChatPanel

Make these properties public so they can be accessed externally for
test automation and other programmatic control.

Changes:
- Change agent, agentInterface, artifactsPanel from private to public

Mario Zechner · 2025-10-10 11:40:07 +02:00

3db2a6fe2c

Refactor JavaScript REPL tool description with consistent pattern

Clean up and restructure JavaScript REPL description following consistent pattern:
- Purpose section
- When to Use section
- Environment section
- Common Libraries section
- Important Notes section
- Example section

Removed buildJavaScriptReplDescription() function - no longer dynamically
injecting runtime provider docs into tool description (will move to system prompt).

Changes:
- Replace JAVASCRIPT_REPL_BASE_DESCRIPTION with JAVASCRIPT_REPL_DESCRIPTION
- Remove JAVASCRIPT_REPL_CHART_EXAMPLE and JAVASCRIPT_REPL_FOOTER
- Remove buildJavaScriptReplDescription() function
- Update javascript-repl tool to use static description
- Simpler, more scannable structure with clear hierarchy

Mario Zechner · 2025-10-10 11:31:11 +02:00

2756f2a974

Improve artifacts tool description to emphasize update command

Add clear decision tree and anti-patterns to prevent LLMs from using
get + rewrite when they should use update for targeted edits.

Changes:
- Add CRITICAL workflow section at top with decision tree
- Emphasize 'update' as PREFERRED for edits (token efficient)
- Mark 'rewrite' as LAST RESORT only
- Add ANTI-PATTERNS section showing wrong approaches
- Clarify use cases for each command
- Add examples emphasizing surgical modifications

Mario Zechner · 2025-10-10 02:32:19 +02:00

3337953b2a

Clarify when to use artifacts runtime functions vs artifacts tool

Make it clear that runtime functions are ONLY for programmatically generated
content (by code), not for content the LLM authors directly.

Changes:
- Add WHEN TO USE and DO NOT USE sections to runtime provider description
- Emphasize that createOrUpdateArtifact is for code-generated content
- Point users to artifacts tool for LLM-authored content like summaries

Mario Zechner · 2025-10-10 02:10:36 +02:00

ab9137542c

Replace hasArtifact with listArtifacts

LLMs don't need to check existence - they can just list all artifacts.
Simpler API that returns all filenames at once.

Changes:
- Replace hasArtifact(filename) with listArtifacts() returning string[]
- Add 'list' action handler that returns all artifact keys
- Update examples in prompt to use listArtifacts()

Mario Zechner · 2025-10-10 01:49:03 +02:00

2e56f77599

Merge createArtifact and updateArtifact into createOrUpdateArtifact

LLMs get confused about when to use create vs update. The single function
automatically detects if the artifact exists and chooses the right operation.

Changes:
- Replace createArtifact/updateArtifact with createOrUpdateArtifact in runtime
- Update handler to check existence and use appropriate command (create/rewrite)
- Simplify prompt documentation and examples

Mario Zechner · 2025-10-10 01:43:36 +02:00

7a859cbe52

Fix abort detection to use message stopReason instead of global isStreaming

When abort happens during tool call streaming, the tool result should show as aborted.
Previously used global isStreaming state which would flip when new messages streamed in,
causing spinner to reappear incorrectly.

Changes:
- Use message.stopReason === "aborted" to detect aborted tool calls
- Create synthetic error result for aborted tool calls in ToolMessage component
- Fix Ollama provider key test to return true (can't know which model to test)
- Add newline before HTML execution logs in artifacts update

Mario Zechner · 2025-10-10 01:40:59 +02:00

961112ff83

HTML artifacts call window.complete() when page loads

- Inject <script>if (window.complete) window.complete();</script> before </html>
- HTML artifacts are long-running and don't time out
- Console logs are sent immediately, completion signals when page is ready
- Removes need for artificial timeout in HTML artifacts

Mario Zechner · 2025-10-09 23:49:22 +02:00

6bc098e57b

Simplify ArtifactsRuntimeProvider constructor

- Take artifactsPanel and agent directly instead of 5 separate function parameters
- Define minimal ArtifactsPanelLike and AgentLike interfaces to avoid circular deps
- Update all call sites (ChatPanel, browser-javascript) to use simplified constructor
- Much cleaner and easier to use

Mario Zechner · 2025-10-09 23:37:14 +02:00

695bcf9b6f

Fix javascript-repl to throw errors on execution failure

- Throw error instead of returning output when result.success is false
- Ensures tool call is marked as failed when code execution fails or is aborted
- Matches browser-javascript error handling pattern

Mario Zechner · 2025-10-09 23:30:40 +02:00

c0ad46fa01

Fix console logging and message routing

Console Logging Improvements:
- Changed ConsoleRuntimeProvider to send logs immediately instead of batching
- Track pending send promises and await them in onCompleted callback
- Ensures REPL gets all logs before execution-complete
- Enables real-time console logging for HTML artifacts

Message Routing Fixes:
- Remove "handled" concept from message routing - broadcast all messages to all providers/consumers
- Change handleMessage return type from Promise<boolean> to Promise<void>
- Add debug logging to RuntimeMessageRouter to trace message flow
- Fix duplicate error logging (window error handler now only tracks errors, doesn't log them)

Output Formatting Consistency:
- Remove [LOG], [ERROR] prefixes from console output in both tools
- Show console logs before error messages
- Use "=> value" format for return values in both javascript-repl and browser-javascript
- Remove duplicate "Error:" prefix and extra formatting

Bug Fixes:
- Fix race condition where execution-complete arrived before console logs
- Fix ConsoleRuntimeProvider blocking execution-complete from reaching consumers
- Remove duplicate console log collection from SandboxedIframe
- Fix return value capture by wrapping user code in async function

Mario Zechner · 2025-10-09 23:20:14 +02:00

4ac774cbbb

Remove debug console.log statements causing duplicate error messages

- console.log('Reporting execution error:', finalError) was logging the error
- This caused duplicate error message in output
- Removed all debug console.log statements from window.complete()
- Error is only shown via execution-error message now

Mario Zechner · 2025-10-09 20:53:32 +02:00

38aaaee968

Display return values and fix duplicate error messages in REPL

- javascript-repl now displays return values with => prefix
- Objects are JSON.stringify'd with formatting
- Removed console.error from wrapper to prevent duplicate error messages
- Error is already captured and sent via execution-error message
- Fixes missing return value display and duplicate error output

Mario Zechner · 2025-10-09 20:51:25 +02:00

8747e01c31

Pass return value through execution-complete message instead of logging

- Return value now passed to window.complete(error, returnValue)
- execution-complete message includes returnValue field
- SandboxResult interface updated to include returnValue
- executionConsumer passes returnValue in resolved promise
- Return values properly captured and available to callers

Mario Zechner · 2025-10-09 20:47:26 +02:00

5fc3857666

Fix REPL timeout with return statements by wrapping user code in function

- User code with return statement was exiting the async IIFE early
- Completion callbacks and window.complete() were never reached
- Now wrap user code in userCodeFunc to capture return value
- Return statement returns from userCodeFunc, not outer IIFE
- Completion callbacks and window.complete() always execute
- Return value is logged to console output
- Fixes 30-second timeout when using return statements in REPL

Mario Zechner · 2025-10-09 20:43:39 +02:00

9a3f01fcb8

Fix message routing by broadcasting to all providers and consumers

- Router was stopping propagation after first handler returned true
- This prevented consumers from seeing messages that providers handled
- executionConsumer never received execution-complete because ConsoleRuntimeProvider handled it first
- Now all providers and consumers receive all messages
- Fixes javascript_repl never completing

Mario Zechner · 2025-10-09 20:41:22 +02:00

fdd4e24246

Fix ConsoleRuntimeProvider blocking execution-complete from reaching executionConsumer

- ConsoleRuntimeProvider was handling execution-complete and returning true
- This stopped message propagation before executionConsumer could handle it
- ExecutionConsumer never got execution-complete, so promise never resolved
- Now ConsoleRuntimeProvider responds but returns false to allow propagation
- Fixes javascript_repl never completing (30s timeout)

Mario Zechner · 2025-10-09 20:39:04 +02:00

3f29ce9cb6

Fix race condition by making window.complete() async and awaiting it

- window.complete() was fire-and-forget, causing execution-complete to arrive before console messages
- This caused sandbox to unregister before console message responses arrived
- Made complete() async and await sendRuntimeMessage()
- SandboxedIframe wrapper now awaits window.complete()
- Ensures all messages are processed before cleanup/unregister
- Fixes 30-second timeout on javascript_repl

Mario Zechner · 2025-10-09 20:31:48 +02:00

5dcb5ecc89

Fix ConsoleRuntimeProvider hanging by calling respond()

- ConsoleRuntimeProvider.handleMessage() was not calling respond()
- This caused sendRuntimeMessage() to hang waiting for response (30s timeout)
- Now properly acknowledges console, execution-complete, and execution-error messages
- Fixes javascript_repl hanging on simple console.log() calls

Mario Zechner · 2025-10-09 20:27:02 +02:00

bb138307b1

Unify completion callback pattern across browser-javascript and javascript-repl

- Remove fallback timeout from ConsoleRuntimeProvider (was causing 2s delays)
- Add completion callback support to SandboxedIframe REPL wrapper
- Call completion callbacks before window.complete() in both success/error paths
- Both browser-javascript and javascript-repl now use identical completion pattern
- Ensures console logs are batched and sent before execution completes

Mario Zechner · 2025-10-09 20:24:20 +02:00

33418d9dea

Simplify ConsoleRuntimeProvider to batch-send logs at completion only

Mario Zechner · 2025-10-09 20:16:32 +02:00

6983ad4eaa

Add onCompleted callback system for guaranteed console log delivery

- Add onCompleted() callback registration in RuntimeMessageBridge
- Modify wrapperFunction to call completion callbacks before returning
- Update ConsoleRuntimeProvider to immediate send + completion batch pattern
- Extract DOWNLOADABLE_FILE_RUNTIME_DESCRIPTION from ATTACHMENTS_RUNTIME_DESCRIPTION
- Logs sent immediately (fire-and-forget), unsent logs batched at completion
- Ensures all console logs arrive before tool execution completes

Mario Zechner · 2025-10-09 20:10:20 +02:00

af0297cd16

Add debug logging to RuntimeMessageRouter and increase API key validation tokens

- Add console.log for user script messages in RuntimeMessageRouter
- Increase ProviderKeyInput maxTokens from 10 to 200 for better validation

Mario Zechner · 2025-10-09 18:57:43 +02:00

b288cd9448

Add runtime bridge architecture and fix HTML escaping

Major refactoring to unify runtime providers across sandbox and user script contexts:

1. Runtime Bridge & Router
   - Add RuntimeMessageBridge for unified messaging abstraction
   - Rename SandboxMessageRouter → RuntimeMessageRouter
   - Router now handles both iframe and user script messages
   - Guard for non-extension environments

2. Provider Refactoring
   - ArtifactsRuntimeProvider: Add offline mode with snapshot fallback
   - AttachmentsRuntimeProvider: Remove returnDownloadableFile (moved to dedicated provider)
   - ConsoleRuntimeProvider: Add message collection, remove lifecycle logic
   - FileDownloadRuntimeProvider: New provider for file downloads

3. HTML Escaping Fix
   - Escape </script> in JSON.stringify output to prevent premature tag closure
   - Applies when injecting provider data into <script> tags
   - JavaScript engine automatically unescapes, no runtime changes needed

4. Function Renaming
   - listFiles → listAttachments
   - readTextFile → readTextAttachment
   - readBinaryFile → readBinaryAttachment
   - returnFile → returnDownloadableFile

5. Updated Exports
   - Export new RuntimeMessageBridge and RuntimeMessageRouter
   - Export FileDownloadRuntimeProvider
   - Update all cross-references

This sets the foundation for reusing providers in browser-javascript tool.

Mario Zechner · 2025-10-09 17:32:45 +02:00

c2793d8017

Add image artifact support with proper binary downloads

- Add ImageArtifact component for displaying images (.png, .jpg, .jpeg, .gif, .webp, .bmp, .ico)
- Images stored as base64, displayed via data URLs
- Download button properly decodes base64 to Uint8Array for valid binary downloads
- Fix Lit ChildPart error: use CSS rotation instead of innerHTML manipulation in collapsible headers

Changes:
- web-ui/src/tools/artifacts/ImageArtifact.ts: New image artifact component
- web-ui/src/tools/artifacts/artifacts.ts: Add "image" file type support
- web-ui/src/tools/renderer-registry.ts: Fix collapsible chevron to use CSS rotation
- web-ui/src/index.ts: Export ImageArtifact

Mario Zechner · 2025-10-09 04:34:42 +02:00

d7d79bd533

Don't access innerHTML like that or lit kills itself.

Mario Zechner · 2025-10-09 04:26:03 +02:00

9687551324

Add artifact message persistence for session reconstruction

- Add ArtifactMessage type as core part of AppMessage union (not CustomMessages)
- ArtifactsRuntimeProvider appends artifact messages on create/update/delete
- MessageList filters out artifact messages (UI display only)
- artifacts.ts reconstructFromMessages handles artifact messages
- Export ARTIFACTS_RUNTIME_PROVIDER_DESCRIPTION from main index
- Fix artifact creation bug: pass filename as title instead of mimeType

Changes:
- web-ui/src/components/Messages.ts: Add ArtifactMessage to BaseMessage union
- web-ui/src/components/MessageList.ts: Skip artifact messages in render
- web-ui/src/components/sandbox/ArtifactsRuntimeProvider.ts: Append messages, fix title parameter
- web-ui/src/ChatPanel.ts: Pass agent.appendMessage callback
- web-ui/src/tools/artifacts/artifacts.ts: Handle artifact messages in reconstructFromMessages
- web-ui/src/index.ts: Export ARTIFACTS_RUNTIME_PROVIDER_DESCRIPTION
- web-ui/example/src/custom-messages.ts: Update message transformer to filter artifacts

Mario Zechner · 2025-10-09 04:07:59 +02:00

4d2ca6ab2a

Fix various sandbox issues.

Mario Zechner · 2025-10-08 22:51:32 +02:00

0eaa879d46

Add ollama dependency and dialog backdrop blur

- Add ollama package to web-ui dependencies for ModelSelector
- Add backdrop blur to SettingsDialog (bg-black/50 backdrop-blur-sm)
- Update mini-lit to 0.1.9 for backdropClassName prop support
- Fix TypeScript errors in ModelSelector (ollama import, parameter types)
- Add backward compatibility methods to SessionsStore (saveSession, loadSession, getLatestSessionId)

Mario Zechner · 2025-10-08 17:49:56 +02:00

91c1dc6475

Fix IndexedDB in-line vs out-of-line key handling

- Check if store has keyPath before calling put()
- If keyPath exists (in-line keys), only pass value: store.put(value)
- If no keyPath (out-of-line keys), pass both: store.put(value, key)
- Apply fix to both set() and transaction.set()
- Fixes DataError when saving sessions with keyPath: 'id'

Mario Zechner · 2025-10-08 16:53:54 +02:00

db34ef01bf

Refactor to Store-based architecture

- Create base Store class with private backend and protected getBackend()
- Add SettingsStore, ProviderKeysStore, SessionsStore
- Each store defines its own schema via getConfig()
- AppStorage now takes stores + backend in constructor
- Remove SessionsRepository (logic moved to SessionsStore)
- Update all consumers to use store API (storage.settings.get/set, storage.providerKeys.get/set)
- Update example app to follow new pattern: create stores, gather configs, create backend, wire
- Benefits: stores own their schema, no circular deps, cleaner separation

Mario Zechner · 2025-10-08 16:41:02 +02:00

0de89a750e

Implement unified storage architecture

- Replace fragmented storage backends with single IndexedDBStorageBackend
- Create multi-store StorageBackend interface (storeName parameter)
- Remove old backends: IndexedDBBackend, LocalStorageBackend, SessionIndexedDBBackend, WebExtensionStorageBackend
- Remove old repositories: ProviderKeysRepository, SessionRepository, SettingsRepository
- Simplify AppStorage to directly expose storage methods (getSetting/setSetting, getProviderKey/setProviderKey)
- Create SessionsRepository for session-specific operations
- Update all consumers to use new simplified API
- Update example app to use new storage architecture
- Benefits: 10GB+ quota (vs 10MB chrome.storage), single database, consistent API

Mario Zechner · 2025-10-08 16:14:29 +02:00

bbbc232c7c

Fix spinner showing on aborted tool calls in javascript-repl and artifacts renderers

When isStreaming is false and there's no result (aborted state), show 'complete' state instead of 'inprogress' to avoid showing the spinner.

Mario Zechner · 2025-10-08 14:43:23 +02:00

b3cce7b400

Add cursor-pointer to collapsible header button

Mario Zechner · 2025-10-08 14:17:48 +02:00

46b1abff1d

Prevent default and stop propagation when clicking artifact pill

This prevents the collapsible header from toggling when the pill is clicked.

Mario Zechner · 2025-10-08 14:17:04 +02:00

9a57aa7e1a

Export ArtifactPill from web-ui index

Mario Zechner · 2025-10-08 14:13:23 +02:00

8e017db924

Make openArtifact public and remove type cast in ArtifactPill

Mario Zechner · 2025-10-08 14:12:52 +02:00

3adc8357b1

Make renderHeader and renderCollapsibleHeader accept string | TemplateResult

- Update renderHeader and renderCollapsibleHeader in renderer-registry.ts to accept `text: string | TemplateResult`
- Remove duplicated renderCollapsibleHeaderWithPill helper in artifacts-tool-renderer.ts
- Update all artifact renderer calls to use renderHeaderWithPill() inline
- Remove all separate pill rendering below headers

This allows artifact pills to be rendered inline with header text without code duplication.

Mario Zechner · 2025-10-08 14:12:02 +02:00

58b9c36268

Add clickable artifact pills to tool renderer

- Create ArtifactPill component (similar to SkillPill)
- Renders filename as clickable pill with FileCode2 icon
- Clicking pill opens artifacts panel and selects that artifact
- Update ArtifactsToolRenderer to accept artifactsPanel reference
- Pass artifactsPanel from ChatPanel to renderer on initialization
- Display artifact pill below header for all commands
- Pill only clickable when artifactsPanel reference is available

Mario Zechner · 2025-10-08 14:01:25 +02:00

547be7ce37

Update artifacts-tool-renderer to use collapsible headers

- All actions except DELETE now use collapsible headers
- CREATE/UPDATE/REWRITE/GET/LOGS: code/output collapsed by default
- DELETE: keeps simple non-collapsible header
- Fix isStreaming parameter usage for proper spinner state
- Add smooth 300ms animation on expand/collapse
- Full header is clickable to toggle collapse state

Mario Zechner · 2025-10-08 13:46:52 +02:00

b3efac4591

Make entire collapsible header clickable

- Changed outer div to button element
- Added w-full and text-left classes
- Now click anywhere on header (icon, text, chevron) to expand/collapse

Mario Zechner · 2025-10-08 13:44:59 +02:00

1405c47b50

Add collapsible tool renderers with animated expand/collapse

- Add renderCollapsibleHeader() to renderer-registry
  - Places chevron on right, spinner on left
  - Toggles between ChevronRight (collapsed) and ChevronDown (expanded)
  - Uses max-h-0/max-h-[2000px] with transition-all for smooth animation
  - Dynamically adds/removes mt-3 to avoid margin when collapsed

- Update javascript-repl renderer to use collapsible sections
  - Code and console output hidden by default
  - Only file attachments remain visible
  - 300ms smooth animation on expand/collapse

- Export renderCollapsibleHeader from web-ui index

Mario Zechner · 2025-10-08 13:38:45 +02:00

f646a29d1a

Refactor artifacts renderer and add Console component

- Extract ArtifactsToolRenderer from ArtifactsPanel into standalone renderer
- Fix ChatPanel to register ArtifactsToolRenderer instead of panel
- Implement command-specific rendering logic (create/update/rewrite/get/logs/delete)
- Create reusable Console component with copy button and autoscroll toggle
- Replace custom console implementation with ExpandableSection and Console
- Fix Lit reactivity for HtmlArtifact logs using spread operator
- Add Lucide icons (FileCode2, ChevronsDown, Lock) for UI consistency
- Follow skill.ts patterns with renderHeader and state handling
- Add i18n strings for all artifact actions and console features

Mario Zechner · 2025-10-08 01:54:50 +02:00

8ec9805112

Make message transformer async, expose i18n.

Mario Zechner · 2025-10-07 17:49:23 +02:00

a8159f504f

Rename ChromeStorageBackend to WebExtensionStorageBackend for cross-browser support

- Rename chrome-storage-backend.ts to web-extension-storage-backend.ts
- Update to use globalThis.browser || globalThis.chrome for Firefox/Chrome compatibility
- Export both names for backward compatibility
- Fix tsc --preserveWatchOutput in web-ui dev script to prevent console clearing

This fixes the "browser is not defined" error in Chrome extensions.

Mario Zechner · 2025-10-06 23:48:43 +02:00

f7878c3c71

Remove debug console.log from ChatPanel

Mario Zechner · 2025-10-06 19:01:11 +02:00

bcd109f2e9

Make mini-lit a peer dependency in pi-web-ui

- Move mini-lit from dependencies to peerDependencies
- Keep in devDependencies for development
- Prevents bundlers from including mini-lit when consuming pi-web-ui
- Consumer (sitegeist) provides mini-lit, esbuild bundles it once
- Fixes duplicate mini-lit bundling issue permanently

Mario Zechner · 2025-10-06 18:59:08 +02:00

33145c5f24

Update package-lock.json after mini-lit version changes

Mario Zechner · 2025-10-06 18:51:34 +02:00

6126380879

Remove debug console.log from ChatPanel

Mario Zechner · 2025-10-06 18:40:41 +02:00

7068c01b44

Clean up browser-extension references from monorepo

- Update README.md to reference sitegeist repo
- Update CLAUDE.md to replace browser-extension with web-ui
- Update packages/web-ui/README.md examples to point to sitegeist
- Remove browser-extension exclude from tsconfig.json
- Remove browser-extension from .claude/settings.local.json permissions

Mario Zechner · 2025-10-06 18:40:04 +02:00

2dbe7771fd

232 Commits