Bugfix/workspace file upload and csv encoding - #1502
Merged
Paul Lizer (paullizer) merged 3 commits intoSep 19, 2026
Merged
Paul Lizer (paullizer) merged 3 commits into
Paul Lizer (paullizer) merged 3 commits into
Conversation
…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.
Contributor
|
nice! thank you for the update! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Linked issue
No associated GitHub issue.
Release Notes & Latest Features
Is this visible to end users?
Is this admin-facing (Admin Settings, governance, deployment, config)?
Should this become a Latest Feature card?
Screenshot needed for the card?
Version bump
config.pyremains at0.261.030because the version was already updated in a prior commit included in this PR.config.pyversion is already updated for this PR; no additional bump is neededversion.txtnot needed becausedeployerswas not changedTesting / validation
python -m py_compilepassed for all changed Python files.python scripts/check_swagger_routes.pypassed for all changed route files.12 passed.test_tabular_csv_ansi_encoding.pytest_chat_upload_group_workspace_handoff.pypythoncheck_xss_sinks.py--full-filepassed for all changed application Python files.git diff --checkpassed.route_backend_chats.py:3806; the line is unchanged relative toorigin/Development.Documentation
v0.261.030Security checklist
@swagger_route(security=get_auth_security()); no new Flask routes were addedsanitize_settings_for_user(); no new settings payloads were added