Skip to content

Filter FILE placeholder hints#553

Open
InkiChang wants to merge 2 commits into
lsdefine:mainfrom
InkiChang:fix-file-placeholder-filter
Open

Filter FILE placeholder hints#553
InkiChang wants to merge 2 commits into
lsdefine:mainfrom
InkiChang:fix-file-placeholder-filter

Conversation

@InkiChang
Copy link
Copy Markdown

@InkiChang InkiChang commented Jun 1, 2026

What

  • Ignore instructional FILE placeholders such as [FILE:filepath] when extracting attachments.
  • Deduplicate extracted FILE paths in the shared chat frontend helper.
  • Reuse the shared extractor in the Feishu frontend.

Why

  • The FILE hint shown to agents can be echoed back literally and was being treated as a missing attachment path in Feishu.
  • Filtering placeholder examples prevents frontends that use the shared extractor from trying to send non-existent files.

Testing

  • python3 -m unittest tests.test_chatapp_common
  • python3 -m py_compile frontends/chatapp_common.py frontends/fsapp.py tests/test_chatapp_common.py

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