Skip to content

fix(coder-labs/gemini): base64-encode task prompt to handle special characters#907

Open
jdomeracki-coder wants to merge 1 commit into
mainfrom
jd/gemini-task-prompt-escaping
Open

fix(coder-labs/gemini): base64-encode task prompt to handle special characters#907
jdomeracki-coder wants to merge 1 commit into
mainfrom
jd/gemini-task-prompt-escaping

Conversation

@jdomeracki-coder
Copy link
Copy Markdown
Contributor

Encodes the Gemini module's task_prompt before passing it into the start script and decodes it in start.sh, matching how gemini_system_prompt is already handled. Prompts containing single quotes or other shell metacharacters are now passed through correctly instead of breaking the start script.

Type of Change

  • Bug fix

Module Information

Path: registry/coder-labs/modules/gemini
New version: v3.0.2
Breaking change: No

Testing & Validation

  • Tests pass (bun test main.test.ts) — added a regression test for prompts containing special characters
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

None


Generated by Coder Agents on behalf of @jdomeracki-coder.

…haracters

Encode task_prompt before passing it into the start script and decode it in
start.sh, matching how the system prompt is already handled. This ensures
prompts containing single quotes or other shell metacharacters are passed
through correctly instead of breaking the start script.

Adds a regression test and bumps the module to v3.0.2.
@jdomeracki-coder jdomeracki-coder requested review from 35C4n0r and DevelopmentCats and removed request for 35C4n0r June 2, 2026 12:55
Copy link
Copy Markdown
Collaborator

@DevelopmentCats DevelopmentCats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I will test locally just to ensure it behaves the same way but I cant see why it wouldn't

@matifali matifali added the version:patch Add to PRs requiring a patch version upgrade label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:patch Add to PRs requiring a patch version upgrade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants