Skip to content

fix(smartypants): sanitize history to avoid provider errors on reasoning and empty content#2835

Closed
paulbettner wants to merge 91 commits into
anomalyco:devfrom
Smarty-Pants-Inc:dev
Closed

fix(smartypants): sanitize history to avoid provider errors on reasoning and empty content#2835
paulbettner wants to merge 91 commits into
anomalyco:devfrom
Smarty-Pants-Inc:dev

Conversation

@paulbettner
Copy link
Copy Markdown

@paulbettner paulbettner commented Sep 28, 2025

Summary

  • Drop prior assistant reasoning parts from history passed to providers (display-only), preventing OpenAI Responses API error: item of type 'reasoning' provided without required following item.
  • Filter aborted assistant messages with no contentful parts (text or completed/error tool results) so history never serializes to empty content, avoiding messages.N must have non-empty content.

Rationale

Interruptions (Ctrl-C) could leave an assistant turn with only step markers or partial tool scaffolding; combined with replaying reasoning chunks, subsequent calls errored and wedged the session. This patch makes history safe while preserving visible reasoning and contentful context.

Scope

  • smartypants: provider transform + message-v2 + prompt filter only.

Notes

  • Typecheck OK via turborepo; no API or configuration changes.

paulbettner and others added 30 commits September 26, 2025 21:10
@paulbettner
core: env-gated Langfuse tracing for Bus events (Node OTEL; Bun v3 fa…
7c18b14 
…llback)

Adds minimal Langfuse integration to observe sessions/events via Bus. Node uses @langfuse/otel span processor; Bun falls back to langfuse v3 client. No effect unless LANGFUSE_* env vars are set.
@paulbettner
observe: instrument message/tool/step parts with per-session tracing …
b0abc68 
…in Bun fallback (Langfuse v3)
@paulbettner
observe: wrap global fetch to trace outgoing HTTP requests (skip Lang…
389d920 
…fuse)
@paulbettner
observe: switch to v4-only (Node OTel); remove v3 client and Bun fall…
a8607df 
…back
@paulbettner
observe: add v4 Node sidecar and auto-start from server; stream SSE e…
bf82c32 
…vents to Langfuse with per-session observations
@paulbettner
observe: auto-start v4 Node sidecar from server (worktree)
0c80b13
@paulbettner
observe: sidecar initOtel robust await of NodeSDK.start()
3de39ee
@paulbettner
observe(sidecar): set correct output field and TOOL type in observati…
21efb8e 
…ons; move tokens/cost to metadata
@paulbettner
observe: attach correct sessionId on all events (use info.sessionID/p…
bf36170 
…art.sessionID); mark assistant as GENERATION
@paulbettner
observe(sidecar): enrich observations
c1320dc 
- type: set GENERATION/TOOL/EVENT/SPAN appropriately
- sanitize text (2k) and data: URLs
- include messageID in metadata
- ensure sessionId from sessionID|info.sessionID|part.sessionID
@paulbettner
observe: fix duplicate variable in v4 Node instrumentation (reuse p)
88e3f1c
@paulbettner
observe(sidecar): aggregate assistant generation I/O
7925fa1 
- collect user input text per session
- accumulate assistant text parts by messageID
- finalize generation output on step-finish with tokens/cost metadata
@paulbettner
Smarty branding: TUI/CLI logos, help banner, brand constant; caret fix (
efcb1e1 
#1)

* Rebrand opencode to smartypants

- Replace all logo SVGs with custom 'smartypants' lettering
- Update React Logo component with new SVG paths and dimensions
- Rename packages/opencode to packages/smartypants
- Update package.json names and binary references
- Update TUI Go main.go with new binary name and env vars
- Update install script to use smartypants naming
- Replace logo references in README.md

Generated 8 logo variants:
- logo-light.svg / logo-dark.svg (main horizontal logos)
- logo-ornate-light.svg / logo-ornate-dark.svg (with transparency effects)
- logomark-light.svg / logomark-dark.svg (SP initials only)
- logo-square-light.svg / logo-square-dark.svg (compact versions)

* Replace with beautiful rainbow ASCII art logos

🌈 Major visual upgrade:
- Replace blocky logos with stunning rainbow gradient ASCII art
- Generated using https://patorjk.com/software/taag/
- Beautiful glowing effect with multiple color themes
- DRY refactored Logo component with reusable ASCII art definitions
- Added logo-generator.js tool to packages/smartypants/tools/

🎨 Features:
- Rainbow gradient with 7-color spectrum
- Mono theme for dark backgrounds
- Scalable ASCII art that looks crisp at any size
- Configurable glow intensity (subtle for 'full', strong for 'ornate')
- Support for mark/full/ornate variants
- Blue, purple, green, red color scheme options

🛠 Technical improvements:
- DRY code with shared ASCII_ART constants
- Reusable gradient and filter definitions
- Consistent scaling and typography
- Proper aspect ratios maintained

* Fix module system and add comprehensive tests

🔧 Module fixes:
- Convert logo-generator.js to logo-generator.cjs for CommonJS compatibility
- Update workspace dependency references
- Clean package naming structure

🧪 Testing suite:
- logo-test.html - Visual gallery of all logo variants
- logo-component-test.tsx - DRY component verification
- Verified logo generator tool produces all variants

✨ Production ready:
- Beautiful rainbow ASCII logos with perfect scaling
- DRY React component architecture
- Multiple themes (rainbow, mono, blue, purple, green, red)
- All variants (mark, full, ornate) working
- Tools for generating additional color schemes

* tui,cli,branding: adopt Smarty logo everywhere; static rainbow + help spacing; fix caret alignment

Switch to consistent user-facing Smarty branding in TUI/CLI and identity assets. Keep static rainbow ASCII (no animation) for stability, preserve help banner spacing, align caret with input, and ensure TUI fallback/env work reliably.

* gitignore: ignore bun.lock

Ignore local Bun lockfile to avoid untracked noise. Removed untracked 'smarty/' directory locally as requested.

* branding: centralize user-facing name and apply across CLI/TUI

Introduce UI.BRAND (env-configurable, default "smarty") and replace user-facing "opencode" strings in help/command descriptions, status bar, and TUI toasts. Do not change internal identifiers or protocol strings; keep server-ready log text unchanged for SDK compatibility.

* cli: add missing UI import for branding strings in serve/attach\n\nEnsure UI.BRAND usage doesn’t throw ReferenceError by importing UI in these modules; no behavioral changes beyond fixing branding display.

* tools: add HTML helpers for multi-color ASCII/SVG logo experiments\n\nCheck in small static pages used to preview and iterate on rainbow logo rendering; no runtime impact.

* app: fix SVG JSX attribute xml:space -> xmlSpace in Logo to satisfy TS types (Solid JSX)

* app: preserve ASCII spacing in SVG text via style='white-space: pre' (drop xmlSpace attr for TSX types)

* app: use camelCase SVG text attributes (fontFamily/fontSize/fontWeight) to satisfy TS types

* app: use hyphenated SVG attributes with string values for Solid types (font-size/font-family/font-weight)
@paulbettner
merge: sync origin/dev into feature/langfuse-observability; adapt sid…
d0565cf 
…ecar autostart to smartypants path and env gate; resolve bun.lock
@paulbettner
observability(langfuse): switch to sidecar-only v4 tracing to reliabl…
378f2f7 
…y capture generation input/output and tools, and log canonical trace URLs; gate in-app tracing behind OPENCODE_OBSERVE=langfuse-app and fix workspace refs.
@paulbettner
fix(types): quiet in-app tracing types under langfuse-app gate and br…
bbf55a0 
…acket-index env; keep sidecar-only as default
@paulbettner
docs: add comprehensive README for Langfuse integration documenting a…
36fb1ca 
…rchitecture, known issues, and development workflow
@paulbettner
Format and improve documentation
280411c 
- Apply consistent formatting to sidecar.mjs
- Improve README formatting and clarity
- Minor formatting fix in server.ts
@paulbettner
Merge pull request #2 from Smarty-Pants-Inc/feature/langfuse-observab…
6847f3b 
…ility

feat: add Langfuse v4 observability via sidecar architecture
@paulbettner
observe(sidecar): dedupe streaming text, surface reasoning, and nest …
b8ec352 
…traces under a session root to simplify debugging; update README to mark issues fixed
@paulbettner
Merge pull request #3 from Smarty-Pants-Inc/fix/langfuse-sidecar-trac…
8cdfc62 
…e-structure

observe(sidecar): dedupe streaming text, surface reasoning, nest under session root
@paulbettner
fix(tui): treat interrupted streams as clean aborts
eb02ca2 
- Map ESC/abort and OpenAI interrupted streaming error to MessageAbortedError ("Request was aborted")
- Preserve tool input/start time when marking aborted tools
@actions-user
chore: format code
e181cfe
@paulbettner
perf(tui): stop idle redraw loops and confine shimmer to latest messa…
9cf31f7 
…ge\n\n- Gate editor spinner ticks to busy/permission states to avoid perpetual idle re-renders\n- Restrict shimmer to the newest in-progress assistant/tool; stop on MessageAbortedError\n- Keeps 90ms cadence while eliminating runaway shimmer after interrupts\n\nMotivation: reduce CPU/heat and TUI slowdowns from idle render churn and aborted sessions. (#4)
@paulbettner
fix(smartypants): sanitize history for provider input by dropping pri…
7eed4da 
…or assistant reasoning parts and filtering aborted assistant messages with no content\n\nPrevents OpenAI Responses API errors (reasoning items in input) and 'messages.N must have non-empty content' when a turn is interrupted. Keeps reasoning for display only; retains contentful aborted messages (text or completed/error tool results).
@paulbettner
Merge pull request #5 from Smarty-Pants-Inc/fix/sanitize-history-reas…
929c2be 
…oning-and-empty-assistant

fix(smartypants): sanitize history to avoid provider errors on reasoning and empty content
@paulbettner
docs+observe+tui: document Smarty Pants fork and land Langfuse/TUI re…
b99648a 
…liability upgrades (#6)

- Add README section for fork enhancements (Langfuse sidecar, TUI link + /trace, run/exit fixes)
- Batch assistant output; emit reasoning only when present; clean sidecar shutdown
- Right‑align Langfuse link with padding; add /trace; fix CLI/TUI explicit exits
- Server: pass actual bound port to sidecar; keep SSE and event logging
@actions-user
chore: format code
b874251
@actions-user
ignore: update download stats 2025-09-28
dd140cd
@paulbettner
tui: scope permission events to active session family; remove cross-s…
52c7c19 
…ession redraws and spinner churn\n\n- Gate permission.updated/replied to active session or parent only\n- Only re-render chat on in-scope permission events\n- Animate editor spinner only for in-scope permissions\n- De-duplicate permission queue by ID\n- Clean permission queue on session switch\n- Stream events scoped by project worktree with fallback\n\nMotivation: eliminate background CPU and animation stutter from unrelated sessions without altering UX. (#7)
paulbettner and others added 28 commits September 29, 2025 02:04
@paulbettner
Merge pull request #10 from Smarty-Pants-Inc/integration/dev-sync-202…
93e4f7d 
…5-09-29

integration: sync downstream/fixes + downstream/smarty into dev
@paulbettner
test: align with new glob/list semantics; file path regex under Confi…
60a1980 
…gMarkdown

- Update fileRegex test to use ConfigMarkdown.FILE_REGEX
- Relax glob basic assertion (count > 0, truncated boolean)
- Point list fixture to packages/smartypants/test/fixtures and update snapshot
- Mark ambiguous replace (multiple matches) as expected failure
- Resolve stray merge artifact in TUI (payload var)
@paulbettner
Merge pull request #11 from Smarty-Pants-Inc/fix/dev-lockfile
d07774f 
test: update smartypants tests for new glob/list + file regex move
@paulbettner
ci(test): make turbo test work and update snapshot
669e937 
- Add  script to packages/smartypants
- Add  task to turbo.json so • Packages in scope: @opencode/app, @opencode/console-app, @opencode/console-core, @opencode/console-function, @opencode/console-resource, @opencode/console-scripts, @opencode/function, @opencode/web, @smartypants-ai/plugin, @smartypants-ai/sdk, opencode-langfuse-sidecar, smartypants
• Running test in 12 packages
• Remote caching disabled
smartypants:test: cache miss, executing ed989ced4798610d
smartypants:test: $ bun test
smartypants:test: bun test v1.2.22 (6bafe260)
smartypants:test:
smartypants:test: test/bun.test.ts:
smartypants:test: (pass) BunProc registry configuration > should not contain hardcoded registry parameters [0.15ms]
smartypants:test: (pass) BunProc registry configuration > should use Bun's default registry resolution [0.09ms]
smartypants:test: (pass) BunProc registry configuration > should have correct command structure without registry [0.10ms]
smartypants:test:
smartypants:test: test/snapshot/snapshot.test.ts:
smartypants:test: (pass) tracks deleted files correctly [207.90ms]
smartypants:test: (pass) revert should remove new files [223.43ms]
smartypants:test: (pass) revert in subdirectory [248.70ms]
smartypants:test: (pass) multiple file operations [307.14ms]
smartypants:test: (pass) empty directory handling [223.87ms]
smartypants:test: (pass) binary file handling [246.09ms]
smartypants:test: (pass) symlink handling [228.33ms]
smartypants:test: (pass) large file handling [236.77ms]
smartypants:test: (pass) nested directory revert [257.52ms]
smartypants:test: (pass) special characters in filenames [203.09ms]
smartypants:test: (pass) revert with empty patches [111.16ms]
smartypants:test: (pass) patch with invalid hash [204.79ms]
smartypants:test: (pass) revert non-existent file [181.31ms]
smartypants:test: (pass) unicode filenames [210.90ms]
smartypants:test: (pass) very long filenames [223.00ms]
smartypants:test: (pass) hidden files [224.85ms]
smartypants:test: (pass) nested symlinks [214.53ms]
smartypants:test: (pass) file permissions and ownership changes [217.78ms]
smartypants:test: (pass) circular symlinks [232.55ms]
smartypants:test: (pass) gitignore changes [226.30ms]
smartypants:test: (pass) concurrent file operations during patch [220.58ms]
smartypants:test: (pass) snapshot state isolation between projects [437.90ms]
smartypants:test: (pass) track with no changes returns same hash [250.23ms]
smartypants:test: (pass) diff function with various changes [184.44ms]
smartypants:test: (pass) restore function [200.45ms]
smartypants:test:
smartypants:test: test/config/markdown.test.ts:
smartypants:test: (pass) should extract exactly 12 file references
smartypants:test: (pass) should extract valid/path/to/a/file
smartypants:test: (pass) should extract another-valid/path/to/a/file
smartypants:test: (pass) should extract paths ignoring comma after
smartypants:test: (pass) should extract a path with a file extension and comma after
smartypants:test: (pass) should extract a path with multiple dots and comma after
smartypants:test: (pass) should extract hidden directory
smartypants:test: (pass) should extract hidden file
smartypants:test: (pass) should extract a file ignoring period at end of sentence
smartypants:test: (pass) should extract an absolute path with an extension
smartypants:test: (pass) should extract an absolute path without an extension
smartypants:test: (pass) should extract an absolute path in home directory
smartypants:test: (pass) should extract an absolute path under home directory
smartypants:test: (pass) should not match when preceded by backtick
smartypants:test: (pass) should not match email addresses
smartypants:test:
smartypants:test: test/config/config.test.ts:
smartypants:test: (pass) loads config with defaults when no files exist [0.91ms]
smartypants:test: (pass) loads JSON config file [2.64ms]
smartypants:test: (pass) loads JSONC config file [0.93ms]
smartypants:test: (pass) merges multiple config files with correct precedence [0.99ms]
smartypants:test: (pass) handles environment variable substitution [0.81ms]
smartypants:test: (pass) handles file inclusion substitution [0.96ms]
smartypants:test: (pass) validates config schema and throws on invalid fields [1.11ms]
smartypants:test: (pass) throws error for invalid JSON [0.80ms]
smartypants:test: (pass) handles agent configuration [1.28ms]
smartypants:test: (pass) handles command configuration [1.02ms]
smartypants:test: (pass) migrates autoshare to share field [0.82ms]
smartypants:test: (pass) migrates mode field to agent field [0.88ms]
smartypants:test: (pass) loads config from .opencode directory [2.40ms]
smartypants:test: (pass) updates config and writes to file [0.70ms]
smartypants:test: (pass) gets config directories [1.42ms]
smartypants:test:
smartypants:test: test/tool/tool.test.ts:
smartypants:test: (pass) tool.glob > truncate [885.74ms]
smartypants:test: (pass) tool.glob > basic [621.82ms]
smartypants:test: 61 |       },
smartypants:test: 62 |     })
smartypants:test: 63 |
smartypants:test: 64 |     // Normalize absolute path to relative for consistent snapshots
smartypants:test: 65 |     const normalizedOutput = result.output.replace(fixturePath, "packages/smartypants/test/fixtures")
smartypants:test: 66 |     expect(normalizedOutput).toMatchSnapshot()
smartypants:test:                                   ^
smartypants:test: error: expect(received).toMatchSnapshot(expected)
smartypants:test:
smartypants:test:
smartypants:test:   "packages/smartypants/test/fixtures/
smartypants:test: -   .devcontainer/
smartypants:test: -     Dockerfile
smartypants:test: -     devcontainer.json
smartypants:test: -     postCreate.sh
smartypants:test: -     postStart.sh
smartypants:test: -   .smarty/
smartypants:test: -     bin/
smartypants:test: -       pants
smartypants:test: -       pants.pyz
smartypants:test: -     tmp/
smartypants:test: -       2025-08-15-mwm-counsel.md
smartypants:test: -       2025-09-05-smartypants-x-sizzle-reel.md
smartypants:test: -       2025-09-23-smarty-pants-onboarding.md
smartypants:test: -       2025-09-23-smarty-pants-onboarding.raw.txt
smartypants:test: -       prompt-composer-README.md
smartypants:test: -   artifacts/
smartypants:test: -     dot-smarty-0.1.0a1.tgz
smartypants:test: -   docker/
smartypants:test: -     smarty-cli.Dockerfile
smartypants:test: -   external/
smartypants:test: -     opencode/
smartypants:test: -       .github/
smartypants:test: -         workflows/
smartypants:test: -           deploy.yml
smartypants:test: -           duplicate-issues.yml
smartypants:test: -           format.yml
smartypants:test: -           guidelines-check.yml
smartypants:test: -           notify-discord.yml
smartypants:test: -           opencode.yml
smartypants:test: -           publish-github-action.yml
smartypants:test: -           publish-vscode.yml
smartypants:test: -           publish.yml
smartypants:test: -           snapshot.yml
smartypants:test: -           stats.yml
smartypants:test: -           test.yml
smartypants:test: -           typecheck.yml
smartypants:test: -       .opencode/
smartypants:test: -         agent/
smartypants:test: -           docs.md
smartypants:test: -           git-committer.md
smartypants:test: -         command/
smartypants:test: -           commit.md
smartypants:test: -           hello.md
smartypants:test: -       packages/
smartypants:test: -         console/
smartypants:test: -           app/
smartypants:test: -             src/
smartypants:test: -               routes/
smartypants:test: -                 debug/
smartypants:test: -                   index.ts
smartypants:test: -                 stripe/
smartypants:test: -                   webhook.ts
smartypants:test: -                 zen/
smartypants:test: -                   v1/
smartypants:test: -                     messages.ts
smartypants:test: -                     responses.ts
smartypants:test: -                   handler.ts
smartypants:test: -                 [...404].css
smartypants:test: -                 [...404].tsx
smartypants:test: -                 discord.ts
smartypants:test: -                 workspace.css
smartypants:test: -                 workspace.tsx
smartypants:test: -           function/
smartypants:test: -             src/
smartypants:test: -               auth.ts
smartypants:test: -               log-processor.ts
smartypants:test: -             package.json
smartypants:test: -             sst-env.d.ts
smartypants:test: -             tsconfig.json
smartypants:test: -           resource/
smartypants:test: -             bun.lock
smartypants:test: -             package.json
smartypants:test: -             resource.cloudflare.ts
smartypants:test: -             resource.node.ts
smartypants:test: -             sst-env.d.ts
smartypants:test: -             tsconfig.json
smartypants:test: -           scripts/
smartypants:test: -             src/
smartypants:test: -               backfill-usage-provider.ts
smartypants:test: -               reset.ts
smartypants:test: -             .gitignore
smartypants:test: -             package.json
smartypants:test: -             sst-env.d.ts
smartypants:test: -             tsconfig.json
smartypants:test: -         function/
smartypants:test: -           src/
smartypants:test: -             api.ts
smartypants:test: -           package.json
smartypants:test: -           sst-env.d.ts
smartypants:test: -           tsconfig.json
smartypants:test: -         identity/
smartypants:test: -           avatar-dark.png
smartypants:test: -           avatar-light.png
smartypants:test: -           logo-dark.svg
smartypants:test: -           logo-light.svg
smartypants:test: -           logo-ornate-dark.svg
smartypants:test: -           logo-ornate-light.svg
smartypants:test: -           logo-square-dark.svg
smartypants:test: -           logo-square-light.svg
smartypants:test: -           logomark-dark.svg
smartypants:test: -           logomark-light.svg
smartypants:test: -         plugin/
smartypants:test: -           src/
smartypants:test: -             example.ts
smartypants:test: -             index.ts
smartypants:test: -             shell.ts
smartypants:test: -             tool.ts
smartypants:test: -           tsconfig.json
smartypants:test: -         sdk/
smartypants:test: -           stainless/
smartypants:test: -             generate.ts
smartypants:test: -             stainless-workspace.json
smartypants:test: -             stainless.yml
smartypants:test: -           .gitignore
smartypants:test: -       patches/
smartypants:test: -         @solidjs%2Fstart@1.1.7.patch
smartypants:test: -       specs/
smartypants:test: -         project.md
smartypants:test: -       AGENTS.md
smartypants:test: -       bun.lock
smartypants:test: -       bunfig.toml
smartypants:test: -       opencode.json
smartypants:test: -       sst-env.d.ts
smartypants:test: -       sst.config.ts
smartypants:test: -       tsconfig.json
smartypants:test: -       turbo.json
smartypants:test: -   .envrc
smartypants:test: -   .gitmodules
smartypants:test: -   .sops.yaml
smartypants:test: -   AGENTS.md
smartypants:test: -   CLAUDE.md
smartypants:test: -   PLAN_OPENCODE.md
smartypants:test: -   README.md
smartypants:test: -   SCRATCH.md
smartypants:test: -   SMARTY_PANTS_MVP.md
smartypants:test: -   logos.txt
smartypants:test: +   example/
smartypants:test: +     broken.ts
smartypants:test: +     cli.ts
smartypants:test: +     ink.tsx
smartypants:test:   "
smartypants:test:
smartypants:test:
smartypants:test: - Expected  - 136
smartypants:test: + Received  + 4
smartypants:test:
smartypants:test:       at <anonymous> (/Users/paulbettner/Projects/smarty-dev/.worktrees/opencode-fixes/packages/smartypants/test/tool/tool.test.ts:66:30)
smartypants:test: (fail) tool.ls > basic [591.46ms]
smartypants:test:
smartypants:test: test/tool/edit.test.ts:
smartypants:test: (pass) EditTool Replacers > case 0 [0.05ms]
smartypants:test: (pass) EditTool Replacers > case 1 [0.12ms]
smartypants:test: (pass) EditTool Replacers > case 2
smartypants:test: (pass) EditTool Replacers > case 3
smartypants:test: (pass) EditTool Replacers > case 4 [0.03ms]
smartypants:test: (pass) EditTool Replacers > case 5 [0.34ms]
smartypants:test: (pass) EditTool Replacers > case 6 [0.05ms]
smartypants:test: (pass) EditTool Replacers > case 7 [0.01ms]
smartypants:test: (pass) EditTool Replacers > case 8 [0.08ms]
smartypants:test: (pass) EditTool Replacers > case 9 [0.03ms]
smartypants:test: (pass) EditTool Replacers > case 10
smartypants:test: (pass) EditTool Replacers > case 11
smartypants:test: (pass) EditTool Replacers > case 12 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 13 [0.30ms]
smartypants:test: (pass) EditTool Replacers > case 14 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 15 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 16 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 17
smartypants:test: (pass) EditTool Replacers > case 18
smartypants:test: (pass) EditTool Replacers > case 19
smartypants:test: (pass) EditTool Replacers > case 20 [0.03ms]
smartypants:test: (pass) EditTool Replacers > case 21
smartypants:test: (pass) EditTool Replacers > case 22
smartypants:test: (pass) EditTool Replacers > case 23
smartypants:test: (pass) EditTool Replacers > case 24
smartypants:test: (pass) EditTool Replacers > case 25
smartypants:test: (pass) EditTool Replacers > case 26
smartypants:test: (pass) EditTool Replacers > case 27
smartypants:test: (pass) EditTool Replacers > case 28 [0.03ms]
smartypants:test: (pass) EditTool Replacers > case 29 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 30
smartypants:test: (pass) EditTool Replacers > case 31
smartypants:test: (pass) EditTool Replacers > case 32
smartypants:test: (pass) EditTool Replacers > case 33
smartypants:test: (pass) EditTool Replacers > case 34
smartypants:test: (pass) EditTool Replacers > case 35
smartypants:test: (pass) EditTool Replacers > case 36
smartypants:test: (pass) EditTool Replacers > case 37
smartypants:test: (pass) EditTool Replacers > case 38 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 39 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 40 [0.02ms]
smartypants:test: (pass) EditTool Replacers > case 41 [0.13ms]
smartypants:test: (pass) EditTool Replacers > case 42
smartypants:test: (pass) EditTool Replacers > case 43
smartypants:test: (pass) EditTool Replacers > case 44 [0.10ms]
smartypants:test: (pass) EditTool Replacers > case 45
smartypants:test: (pass) EditTool Replacers > case 46
smartypants:test: (pass) EditTool Replacers > case 47 [0.03ms]
smartypants:test:
smartypants:test: test/tool/bash.test.ts:
smartypants:test: (pass) tool.bash > basic [12.42ms]
smartypants:test: (pass) tool.bash > cd ../ should fail outside of project root [1.67ms]
smartypants:test:
smartypants:test: test/session/fileRegex.test.ts:
smartypants:test: (pass) fileRegex > should extract exactly 12 file references
smartypants:test: (pass) fileRegex > should extract valid/path/to/a/file [0.24ms]
smartypants:test: (pass) fileRegex > should extract another-valid/path/to/a/file [0.01ms]
smartypants:test: (pass) fileRegex > should extract paths ignoring comma after
smartypants:test: (pass) fileRegex > should extract a path with a file extension and comma after
smartypants:test: (pass) fileRegex > should extract a path with multiple dots and comma after
smartypants:test: (pass) fileRegex > should extract hidden directory
smartypants:test: (pass) fileRegex > should extract hidden file
smartypants:test: (pass) fileRegex > should extract a file ignoring period at end of sentence
smartypants:test: (pass) fileRegex > should extract an absolute path with an extension
smartypants:test: (pass) fileRegex > should extract an absolute path without an extension
smartypants:test: (pass) fileRegex > should extract an absolute path in home directory
smartypants:test: (pass) fileRegex > should extract an absolute path under home directory
smartypants:test: (pass) fileRegex > should not match when preceded by backtick
smartypants:test: (pass) fileRegex > should not match email addresses [0.02ms]
smartypants:test:
smartypants:test: 1 tests failed:
smartypants:test: (fail) tool.ls > basic [591.46ms]
smartypants:test:
smartypants:test:  125 pass
smartypants:test:  1 fail
smartypants:test:  snapshots: 1 failed
smartypants:test:  187 expect() calls
smartypants:test: Ran 126 tests across 8 files. [8.05s]
smartypants:test: error: script "test" exited with code 1

 Tasks:    0 successful, 1 total
Cached:    0 cached, 1 total
  Time:    8.197s
Failed:    smartypants#test succeeds in CI
- Update tool.ls snapshot for new fixture path
@paulbettner
Merge pull request #12 from Smarty-Pants-Inc/fix/dev-lockfile
96c2ebb 
ci: enable turbo test + snapshot update
@paulbettner
test(ci): stabilize list snapshot on fixtures/example and set develop…
9c80879 
…ment conditions for workspace imports

- Run tests with --conditions=development so @smartypants-ai/sdk resolves to TS sources
- Point list test to fixtures/example for consistent tree; update snapshot
@paulbettner
Merge pull request #13 from Smarty-Pants-Inc/fix/dev-lockfile
9d697d0 
ci(test): fix workspace resolution + stable fixtures snapshot
@paulbettner
test(ci): avoid brittle directory tree snapshot; assert count/truncat…
93d5a45 
…ed only for ListTool
@paulbettner
Merge pull request #14 from Smarty-Pants-Inc/fix/dev-lockfile
335f73d 
ci(test): stabilize ListTool test (no snapshot)
@paulbettner
ci(typecheck): resolve @smartypants-ai/sdk via exports by adding cust…
c8af608 
…omConditions=['development'] to plugin tsconfig
@paulbettner
Merge pull request #15 from Smarty-Pants-Inc/fix/dev-lockfile
f533d35 
ci(typecheck): resolve workspace SDK in plugin via customConditions
@paulbettner
ci(typecheck): add customConditions=['development'] + bundler resolut…
039d272 
…ion to app tsconfig for SDK workspace exports
@paulbettner
Merge pull request #16 from Smarty-Pants-Inc/fix/dev-lockfile
b7fa23c 
ci(typecheck): enable development condition in app for SDK exports
@paulbettner
ci(typecheck): resolve workspace packages in smartypants via bundler …
c4c1e32 
…resolution + development condition
@paulbettner
Merge pull request #17 from Smarty-Pants-Inc/fix/dev-lockfile
0116975 
ci(typecheck): resolve smartypants workspace imports via bundler resolution
@paulbettner
ci: skip snapshot/deploy on forks (run only on sst/opencode)
5fca43e
@paulbettner
Merge pull request #18 from Smarty-Pants-Inc/fix/dev-lockfile
fe3b0a1 
ci: skip snapshot/deploy workflows on forks
@paulbettner
ci: make guidelines-check valid and skip on forks (workflow_dispatch …
5bd517d 
…only)
@paulbettner
Merge pull request #19 from Smarty-Pants-Inc/fix/dev-lockfile
77b9a5a 
ci: disable guidelines-check on forks and make workflow valid
@actions-user
ignore: update download stats 2025-09-29
51516fd
@paulbettner
Merge remote-tracking branch 'upstream/dev' into downstream/fixes
da3ac9e
@paulbettner
merge(upstream/dev): resolve conflicts
4205e22 
- STATS.md: take upstream
- packages/smartypants/package.json: keep downstream
@paulbettner
merge(branch): bring downstream/fixes into integration/dev-merge-2025…
010ee7f 
…-09-29 (resolve lockfile/manifests/STATS)
@paulbettner
merge(branch): bring downstream/smarty into integration/dev-merge-202…
3f209b7 
…5-09-29 (prefer dev tests/lockfile)
@actions-user
chore: format code
0f2a9c6
@paulbettner
Merge pull request #22 from Smarty-Pants-Inc/integration/dev-merge-20…
bcce045 
…25-09-29

integration: merge downstream/fixes + downstream/smarty into dev (conflicts resolved)
@paulbettner
docs: local wrapper commands (fork usage)
e8a6183
@paulbettner
Merge pull request #24 from Smarty-Pants-Inc/docs/wrappers-v2
9393594 
docs: local wrapper commands (smarty-dev integration) v2
@paulbettner
Copy link
Copy Markdown
Author

paulbettner commented Sep 30, 2025

Closing to recreate as a clean PR without fork-specific changes (smartypants branding, langfuse integration, etc). Will reopen with only the history sanitization fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@paulbettner @actions-user