Skip to content

fix: center files state and suppress emojis - #36

Merged
iamjr15 merged 1 commit into
mainfrom
fix/files-empty-state-and-response-style
Jul 20, 2026
Merged

fix: center files state and suppress emojis#36
iamjr15 merged 1 commit into
mainfrom
fix/files-empty-state-and-response-style

Conversation

@iamjr15

@iamjr15 iamjr15 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Centers the Files empty-state mark within the editor surface instead of the full iframe viewport, so the explorer width no longer shifts it left.
  • Strengthens the agent narration contract so user-visible responses omit emojis unless the user explicitly requests them.

Decisions Made

Decision Choice Reasoning
Empty-state positioning Absolute positioning inside the editor empty-state container The container already owns the editor bounds; viewport-fixed positioning incorrectly included the explorer pane.
Emoji handling System-prompt invariant with explicit rewriting guidance Covers headings, status lines, templates, skills, and sourced content without changing rendering behavior.

Edge Cases Handled

Scenario Handling
Explorer width changes The mark remains centered against the editor pane.
Skill or template contains emoji The agent rewrites it to plain text unless emojis were requested.

How to Review

  1. Review packages/preview-bridge/src/index.ts for the positioning scope.
  2. Review packages/agent-core/src/mastra/system-prompt.ts for the response-style invariant.

Validation

  • pnpm lint
  • pnpm turbo typecheck
  • pnpm build
  • Direct agent-browser QA in the embedded Files tab with an empty folder
  • Direct agent-browser QA of a fresh celebratory response; zero Extended_Pictographic characters
  • Browser console checked with no new application errors

@vercel

vercel Bot commented Jul 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
cheatcode Ignored Ignored Jul 20, 2026 5:46am

@iamjr15
iamjr15 merged commit ac39dd9 into main Jul 20, 2026
3 checks passed
@iamjr15
iamjr15 deleted the fix/files-empty-state-and-response-style branch July 20, 2026 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant