Skip to content

fix: default function code updates to functionRootPath#524

Merged
binggg merged 2 commits intomainfrom
feature/function-update-default-path
Apr 14, 2026
Merged

fix: default function code updates to functionRootPath#524
binggg merged 2 commits intomainfrom
feature/function-update-default-path

Conversation

@binggg
Copy link
Copy Markdown
Member

@binggg binggg commented Apr 14, 2026

Summary

  • make manageFunctions steer normal code updates toward functionRootPath instead of defaulting to zip/base64 mental models
  • add a fast-fail message when updateFunctionCode is called without either functionRootPath or zipFile
  • update demo prompts, generated tool metadata, docs, and schema assertions to reinforce the default path

Validation

  • cd mcp && npm run build
  • node scripts/generate-tools-json.mjs
  • node scripts/generate-tools-doc.mjs
  • cd mcp && ./node_modules/.bin/vitest run --config vitest.config.js ../tests/function-layer-tools.test.js

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@binggg binggg merged commit 60841d3 into main Apr 14, 2026
3 checks passed
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