Skip to content

[Fix] Fixed console semantic syntax highlighting and top level autocompletion in the global env#17

Merged
Fred-Wu merged 4 commits into
mainfrom
global-env-obj-autocomplete
Jun 16, 2026
Merged

[Fix] Fixed console semantic syntax highlighting and top level autocompletion in the global env#17
Fred-Wu merged 4 commits into
mainfrom
global-env-obj-autocomplete

Conversation

@Fred-Wu

@Fred-Wu Fred-Wu commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fixed live R session completions for global objects

  • Fixed console semantic syntax highlighting to use the active VS Code theme, including auto-detected preferred themes.

  • Removed hard-coded theme extension path discovery and now relies on VS Code’s extension contribution registry.

  • Updated the console R language-server helper to use self$get_workspace(...) instead of stale self$workspace access.

  • Updated changelog and bumped package version to 0.2.8.

Fred Wu added 4 commits June 16, 2026 21:04
- Resolve console syntax colors from the active VS Code theme, including auto-detected preferred themes
- Use VS Code extension contributions for theme lookup instead of hard-coded extension paths
- Update console language server workspace access for current languageserver internals
- Avoid stale token colors after autocomplete edits
@Fred-Wu Fred-Wu marked this pull request as ready for review June 16, 2026 12:55
@Fred-Wu Fred-Wu merged commit f958285 into main Jun 16, 2026
12 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c3d0827100

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +270 to 272
const response = await this.postToSessionServer({ type: "workspace" });
if (!this.isWorkspaceData(response)) {
return;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restore workspace.json loading for session data

This replaces the watched workspace.json read with a POST for {type:"workspace"}, but the vscode-R session server is optional and its web server handles member-completion requests rather than workspace snapshots. In sessions where request.server is null/absent or the server returns null for this request, updateWorkspace() now returns without reading the workspace.json file that workspace.lock is watching, so workspaceData remains undefined and global-environment completions/search-path sync stop working. Please keep the file-based workspace.json path as the source or at least as a fallback.

Useful? React with 👍 / 👎.

@Fred-Wu Fred-Wu deleted the global-env-obj-autocomplete branch June 17, 2026 03:27
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