fix: update tool-call extraction logic for Pi session JSONL

Refine parsing of assistant tool calls in review.sh to handle "toolCall" and "arguments" structures, aligning with Pi session format changes.

action.yml

@@ -18,6 +18,10 @@ inputs:
     description: "Custom API base URL (for OpenAI-compatible providers). Ignored for built-in providers."
     required: false
     default: ""
+  server_url:
+    description: "Override the Gitea/GitHub server URL used for API calls (e.g. http://server:3000 for internal routing inside the runner). Defaults to the auto-detected server URL."
+    required: false
+    default: ""
   token:
     description: "Git platform token for posting PR comments"
     required: true

entrypoint.sh

@@ -11,6 +11,7 @@ export PI_API_KEY="${INPUT_API_KEY}"
 export PI_PROVIDER="${INPUT_PROVIDER:-zai}"
 export PI_MODEL="${INPUT_MODEL:-glm-5.1}"
 export PI_BASE_URL="${INPUT_BASE_URL:-}"
+export PI_SERVER_URL="${INPUT_SERVER_URL:-}"
 export PI_TOOLS="${INPUT_TOOLS:-read,grep,find,ls}"
 export PI_REVIEW_PROMPT="${INPUT_REVIEW_PROMPT:-}"
 export PI_EXCLUDE="${INPUT_EXCLUDE_PATTERNS:-*.lock package-lock.json yarn.lock pnpm-lock.yaml *.min.js *.min.css *.map}"

scripts/review.sh

@@ -86,7 +86,12 @@ REPO="${GITHUB_REPOSITORY:-${GITEA_REPOSITORY:-}}"
 
 # Detect Gitea vs GitHub. Gitea sets $GITHUB_SERVER_URL too, so the only
 # reliable signal is $GITEA_ACTIONS (or a non-github.com server URL).
-SERVER_URL="${GITHUB_SERVER_URL:-${GITEA_SERVER_URL:-}}"
+#
+# $PI_SERVER_URL (server_url input) overrides the auto-detected server URL.
+# Gitea reports its *external* hostname (e.g. git.example.com), which may not
+# be routable from inside the runner's container network. Setting server_url
+# to the internal address (e.g. http://server:3000) fixes API calls.
+SERVER_URL="${PI_SERVER_URL:-${GITHUB_SERVER_URL:-${GITEA_SERVER_URL:-}}}"
 if [ -n "${GITEA_ACTIONS:-}" ] || { [ -n "$SERVER_URL" ] && [ "${SERVER_URL#*github.com}" = "$SERVER_URL" ]; }; then
   API_BASE="${SERVER_URL}/api/v1"
   PLATFORM="gitea"
@@ -250,14 +255,21 @@ if [ ! -s /tmp/pi-review.md ]; then
   exit 1
 fi
 
+echo "Debug mode: PI_DEBUG=${PI_DEBUG}"
+
 # If debug mode, extract tool calls from session and append to review
 if [ "${PI_DEBUG}" = "true" ]; then
+  echo "Session dir contents:"
+  find /tmp/pi-session -type f 2>/dev/null || echo "  (none)"
   SESSION_FILE=$(find /tmp/pi-session -name '*.jsonl' -type f 2>/dev/null | head -1)
+  echo "Session file: ${SESSION_FILE:-<none found>}"
   TOOL_LOG=""
 
   if [ -n "${SESSION_FILE}" ]; then
-    # Extract tool-use entries: each line is a JSON object.
-    # Tool calls appear as messages with tool_use/function_call content.
+    # Extract tool-call entries from Pi session JSONL.
+    # Pi stores assistant messages with content blocks of type "toolCall"
+    # (NOT the Anthropic SDK's "tool_use"), using "arguments" (NOT "input").
+    # Tool results are separate entries with role "toolResult".
     TOOL_LOG=$(node -e "
       const fs = require('fs');
       const lines = fs.readFileSync('${SESSION_FILE}', 'utf8').trim().split('\n');
@@ -265,18 +277,16 @@ if [ "${PI_DEBUG}" = "true" ]; then
       for (const line of lines) {
         try {
           const entry = JSON.parse(line);
-          if (entry.type === 'message') {
-            const msg = entry.message;
-            // Model requesting tool use
-            if (msg.role === 'assistant' && msg.content) {
-              const parts = Array.isArray(msg.content) ? msg.content : [msg.content];
-              for (const part of parts) {
-                if (typeof part === 'object' && part.type === 'tool_use') {
-                  const input = part.input || {};
-                  const args = Object.entries(input).map(([k,v]) => k + '=' + JSON.stringify(v)).join(' ');
-                  entries.push('[tool:' + part.name + '] ' + args);
-                }
-              }
+          if (entry.type !== 'message') continue;
+          const msg = entry.message;
+          if (!msg || !msg.content) continue;
+          const parts = Array.isArray(msg.content) ? msg.content : [msg.content];
+          for (const part of parts) {
+            if (typeof part !== 'object') continue;
+            if (part.type === 'toolCall') {
+              const args = Object.entries(part.arguments || {})
+                .map(([k,v]) => k + '=' + JSON.stringify(v)).join(' ');
+              entries.push('[tool:' + part.name + '] ' + args);
             }
           }
         } catch {}