Skip to content

[Repo Assist] fix: null activeTextEditor crash, per-document settings, \$\{workspaceFolder} in standard#47

Merged
soderlind merged 1 commit intomasterfrom
repo-assist/fix-null-activetexteditor-and-standard-path-4cf8aad931db1b2f
Mar 9, 2026
Merged

[Repo Assist] fix: null activeTextEditor crash, per-document settings, \$\{workspaceFolder} in standard#47
soderlind merged 1 commit intomasterfrom
repo-assist/fix-null-activetexteditor-and-standard-path-4cf8aad931db1b2f

Conversation

@soderlind
Copy link
Copy Markdown
Owner

…ath vars

Bump version to 0.0.9.

…ath vars

- loadSettings() no longer crashes when window.activeTextEditor is null,
  fixing extension activation errors (#35, #43)
- loadSettings() now accepts an optional uri parameter so it can be
  scoped to the document being formatted
- format() calls loadSettings(document.uri) on every invocation,
  ensuring per-folder settings in multi-root workspaces are honoured (#36)
- Resolve ${workspaceFolder} and ${workspaceRoot} in phpcbf.standard (#38)
- onWillSaveTextDocument uses event.document.uri instead of
  window.activeTextEditor to avoid potential null-reference errors

Bump version to 0.0.9.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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