Skip to content

fix(agent): pass uploaded file urls to tools#7091

Open
FinleyGe wants to merge 1 commit into
labring:mainfrom
FinleyGe:codex/agent-file-url-tool-params
Open

fix(agent): pass uploaded file urls to tools#7091
FinleyGe wants to merge 1 commit into
labring:mainfrom
FinleyGe:codex/agent-file-url-tool-params

Conversation

@FinleyGe

Copy link
Copy Markdown
Collaborator

Summary

  • expose uploaded file type and url in agent file reminders
  • keep fileUrlMap for all uploaded files while preserving document-only filesMap for read_files
  • replace exact agent file ids in user tool params with URLs for both agent and agentv2

Tests

  • @fastgpt/service test run: 169 passed, 2 skipped
  • git diff --check

Notes

Draft PR for discussion. Current implementation uses the uploaded file previewUrl; refreshing expired historical signed URLs from stored key can be handled separately.

@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 14.81% 1221 / 8244
🔵 Statements 14.76% 1277 / 8646
🔵 Functions 12.95% 256 / 1976
🔵 Branches 12.44% 564 / 4531
File CoverageNo changed files found.
Generated in workflow #754 for commit 687a881 by the Vitest Coverage Report Action

@FinleyGe FinleyGe requested a review from c121914yu June 10, 2026 10:22
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

Admin Preview Image Ready!

ghcr.io/labring/fastgpt-pr:admin_687a8812c9b414c0c76fc3ee9f84f72b834cdfbd

🕒 Time: 2026-06-11 17:49:14 (UTC+8)

@FinleyGe FinleyGe marked this pull request as ready for review June 11, 2026 09:37
@FinleyGe FinleyGe force-pushed the codex/agent-file-url-tool-params branch from 5df0472 to 687a881 Compare June 11, 2026 09:38
@github-actions

Copy link
Copy Markdown

Build Successful - Preview fastgpt Image for this PR:

ghcr.io/labring/fastgpt-pr:fastgpt_687a8812c9b414c0c76fc3ee9f84f72b834cdfbd

🕒 Time: 2026-06-11 17:49:27 (UTC+8)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant