Skip to content

Bugfix/workspace file upload and csv encoding - #1502

Merged
Paul Lizer (paullizer) merged 3 commits into
microsoft:Developmentfrom
Xeelee33:bugfix/workspace_file_upload_csv_encoding
Sep 19, 2026
Merged

Paul Lizer (paullizer) merged 3 commits into
microsoft:Developmentfrom
Xeelee33:bugfix/workspace_file_upload_csv_encoding

Conversation

@Xeelee33

Copy link
Copy Markdown
Contributor

Summary

  • Added shared CSV decoding with UTF-8, UTF-8 BOM, Windows-1252, and Latin-1 fallbacks so ANSI-encoded uploads preserve their content during metadata extraction, indexing, citations, and tabular queries.
  • Improved tabular analysis recovery by detecting repeated equivalent tool failures and routing subsequent model attempts toward a different query path instead of repeating the same failing call.
  • Added visible retry lifecycle reporting and server-side diagnostics for repeated tabular tool failures.
  • Fixed group workspace chat uploads so workspace owners are recognized correctly.

Linked issue

No associated GitHub issue.

Release Notes & Latest Features

  • New Feature
  • Bug Fix
  • UI Enhancement
  • Breaking Change
  • Internal only

Is this visible to end users?

  • Yes
  • No

Is this admin-facing (Admin Settings, governance, deployment, config)?

  • Yes
  • No

Should this become a Latest Feature card?

  • Yes
  • No
  • Already added

Screenshot needed for the card?

  • Yes
  • No
  • Attached

Version bump

config.py remains at 0.261.030 because the version was already updated in a prior commit included in this PR.

  • config.py version is already updated for this PR; no additional bump is needed
  • version.txt not needed because deployers was not changed

Testing / validation

  • python -m py_compile passed for all changed Python files.
  • python scripts/check_swagger_routes.py passed for all changed route files.
  • Route policy tests passed: 12 passed.
  • Focused functional tests passed:
    • test_tabular_csv_ansi_encoding.py
    • test_chat_upload_group_workspace_handoff.py
  • python check_xss_sinks.py --full-file passed for all changed application Python files.
  • git diff --check passed.
  • The broken-access-control scan reported one pre-existing finding in route_backend_chats.py:3806; the line is unchanged relative to origin/Development.
  • Tested re-uploading a previously failing .csv file with ANSI encoding/characters and was able to successfully prompt against it.
  • Tested dropping a file into a chat window with the conversation set to a group workspace context as an Admin, Document Manager, and User. Admin and Document Manager could add the file properly, User was unable to add the file.

Documentation

  • Release notes updated under v0.261.030
  • Feature documentation updated, not needed
  • Fix documentation updated

Security checklist

  • New Flask routes include @swagger_route(security=get_auth_security()); no new Flask routes were added
  • Settings sent to non-admin frontends use sanitize_settings_for_user(); no new settings payloads were added
  • Browser JavaScript is served from local SimpleChat static assets only; no browser JavaScript changes were included
  • No secrets, keys, connection strings, or local-only artifacts are included

Xeelee33 and others added 3 commits September 18, 2026 21:49
…agers.

Include group user roles in the chat bootstrap payload so owners, admins, and document managers are correctly recognized as eligible group uploaders. Add regression coverage, documentation, and release notes
Add shared CSV decoding with UTF-8, UTF-8 BOM, Windows-1252, and
Latin-1 fallbacks across tabular processing paths.

Preserve the broader automatic tool-invocation budget while detecting
repeated equivalent failures, emitting visible retry lifecycle updates,
and instructing subsequent model attempts to use a different query path.
@paullizer
Paul Lizer (paullizer) merged commit a46eb81 into microsoft:Development Sep 19, 2026
1 check passed
@paullizer

Copy link
Copy Markdown
Contributor

nice! thank you for the update!

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.

2 participants