Skip to content

Release v1.15.2#3326

Merged
nighca merged 7 commits into
mainfrom
dev
Jul 3, 2026
Merged

Release v1.15.2#3326
nighca merged 7 commits into
mainfrom
dev

Conversation

@nighca

@nighca nighca commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

[skip review]

Deployment Notes

#3303 Split app env ownership

When this change is released, update the Vercel configuration for the xbuilder app:

  • Rename the existing NODE_ENV environment variable to VITE_MODE and set it to production-cn where needed.
  • Remove the other existing environment variables so the build uses the app-specific .env.* files committed in this project.

dependabot Bot and others added 7 commits June 29, 2026 16:39
…up (#3320)

Bumps the infrastructure group with 1 update: [actions/cache](https://github.com/actions/cache).


Updates `actions/cache` from 5 to 6
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v5...v6)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: infrastructure
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps): bump spx to 2.0.4

* chore(deps): bump xgolsw
Normalize LSP completion results to `CompletionList` so array results
keep existing complete-list behavior while incomplete lists can trigger
a fresh request during active completion.

Fixes #3256

Signed-off-by: Aofei Sheng <aofei@aofeisheng.com>
* Add account user CSV import

* Address account import review feedback

* Address latest account import review

* Use file-saver for import CSV example

* Skip conflicting account user imports

* Inline account import conflict check

* Simplify account import CSV modal state

* Preserve CSV import source line numbers
Use `getStringLengthInCodePoints` for frontend logic that must preserve
exact Unicode code point budgets, such as Copilot inputs, AI description
payload construction, and spx model string validation.

Keep ordinary form validation on native JavaScript and browser string
length behavior, while keeping shared API limits in API modules.

Document when frontend code should use native string length and when it
should count Unicode code points.

Signed-off-by: Aofei Sheng <aofei@aofeisheng.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors environment variable handling to separate app-specific configurations, introduces Unicode code point length validation, adds a CSV-based user import feature for account administration, and updates the spx version to 2.0.4. Feedback on these changes highlights a missing import for projectDisplayNameMaxLength in EditorProjectDisplayName.vue that will cause compilation errors. Additionally, improvements are suggested to make CSV parsing more robust by normalizing headers and username casing, and to optimize getStringLengthInCodePoints by using a for...of loop to avoid unnecessary array allocations.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread spx-gui/src/components/editor/navbar/EditorProjectDisplayName.vue
Comment thread spx-gui/src/components/account/admin/account-user-import/csv.ts
Comment thread spx-gui/src/components/account/admin/account-user-import/csv.ts
Comment thread spx-gui/src/components/account/admin/account-user-import/csv.ts
Comment thread spx-gui/src/utils/utils.ts
@nighca nighca merged commit fe28d3c into main Jul 3, 2026
13 checks passed
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