fix: URL-encode function name in fetchFunctionLogs to fix 404 for zero-config functions

[kfirstri]

Note

Description

This PR fixes a bug where fetching logs for zero-config functions with slashes in their names (e.g., stam/foo from functions/stam/foo/entry.ts) resulted in a 404 error. The function name was being interpolated directly into the URL path, causing the slash to be treated as a path separator by the server. The fix applies encodeURIComponent() to functionName in fetchFunctionLogs, consistent with how deploySingleFunction and deleteSingleFunction already handle this. The test server mocks are updated accordingly.

Related Issue

Fixes #421

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)
• Other (please describe):

Changes Made

• Wrap functionName with encodeURIComponent() when building the URL in fetchFunctionLogs (packages/cli/src/core/resources/function/api.ts)
• Update mockFunctionLogs and mockFunctionLogsError in TestAPIServer.ts to use encodeURIComponent(functionName) so mock routes match the encoded URLs
• Brings fetchFunctionLogs in line with deploySingleFunction and deleteSingleFunction, which already apply this encoding

Testing

• I have tested these changes locally
• I have added/updated tests as needed
• All tests pass (npm test)

Checklist

• My code follows the project's style guidelines
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation (if applicable)
• My changes generate no new warnings
• I have updated docs/ (AGENTS.md) if I made architectural changes

Additional Notes

The fix is minimal and consistent with the encoding pattern already used by deploySingleFunction and deleteSingleFunction in the same file. No documentation or architectural changes are needed.

---

_{🤖 Generated by Claude | 2026-03-19 14:30 UTC}

[github-actions]

🚀 Package Preview Available!

---

Install this PR's preview build with npm:

npm i @base44-preview/cli@0.0.47-pr.429.4d0c680

Prefer not to change any import paths? Install using npm alias so your code still imports base44:

npm i "base44@npm:@base44-preview/cli@0.0.47-pr.429.4d0c680"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "base44": "npm:@base44-preview/cli@0.0.47-pr.429.4d0c680"
  }
}

• 📦 Preview Package: @base44-preview/cli@0.0.47-pr.429.4d0c680
• 🔗 View this commit on GitHub

---

_{Preview published to npm registry — try new features instantly!}

[kfirstri]

https://github.com/base44-dev/apper/pull/5772 we need to merge this first

[kfirstri]

somehow this was merged but not closed..