Skip to content

Insert file stub snippet when creating a blank client-side class or routine#1681

Merged
isc-bsaviano merged 1 commit intointersystems-community:masterfrom
isc-bsaviano:fix-1667
Dec 10, 2025
Merged

Insert file stub snippet when creating a blank client-side class or routine#1681
isc-bsaviano merged 1 commit intointersystems-community:masterfrom
isc-bsaviano:fix-1667

Conversation

@isc-bsaviano
Copy link
Contributor

This PR fixes #1667. The new behavior when a blank client-side class or routine is created is that a snippet containing a document stub is inserted into the document. The user can then type whatever name they like and then tab to a comment inside the empty body to start editing. The name does not have a default value, so the document won't be synced to the server with a potentially incorrect generated name. An example of how this works can be seen below. I also removed the objectscript.autoAdjustName setting so documents created with content already existing will never be modified to use a generated name.

issue1667.mov

Copy link
Contributor

@gjsjohnmurray gjsjohnmurray left a comment

Choose a reason for hiding this comment

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

Looks OK. I rarely use client side editing, but am going to approve this PR.

@isc-bsaviano isc-bsaviano merged commit 8a62e44 into intersystems-community:master Dec 10, 2025
5 checks passed
@isc-bsaviano isc-bsaviano deleted the fix-1667 branch December 10, 2025 19:28
ricxJr added a commit to consistem/vscode-objectscript that referenced this pull request Jan 12, 2026
* Prevent users from accidentally opening multiple copies of the same class or routine (intersystems-community#1666)

* Fix opening server-side generated INT routines (intersystems-community#1668)

* Fix running unit tests from a root test item (intersystems-community#1669)

* Bump js-yaml from 4.1.0 to 4.1.1 (intersystems-community#1670)

Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.1.0 to 4.1.1.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](nodeca/js-yaml@4.1.0...4.1.1)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 4.1.1
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump glob from 11.0.1 to 11.1.0 (intersystems-community#1673)

Bumps [glob](https://github.com/isaacs/node-glob) from 11.0.1 to 11.1.0.
- [Changelog](https://github.com/isaacs/node-glob/blob/main/changelog.md)
- [Commits](isaacs/node-glob@v11.0.1...v11.1.0)

---
updated-dependencies:
- dependency-name: glob
  dependency-version: 11.1.0
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix automatic refresh of server-side documents after save and compile (intersystems-community#1678)

* Fix `Show Plan` for IRIS 2026.1+ (intersystems-community#1679)

* Insert file stub snippet when creating a new class or routine using client-side editing (intersystems-community#1681)

* chore: `npm audit fix` (intersystems-community#1682)

* Trigger server-side source control `OpenedDocument` UserAction after a project is modified (intersystems-community#1685)

* Remove barely used configuration settings (intersystems-community#1683)

* Prepare 3.4.0 release (intersystems-community#1692)

* auto bump version with release

* Add `Func` suffix to `Copy Invocation` result for Queries (intersystems-community#1695)

* Fix extension activation when clicking on InterSystems view container in an empty workspace folder (intersystems-community#1694)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Brett Saviano <bsaviano@intersystems.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: John Murray <johnm@georgejames.com>
Co-authored-by: ProjectBot <bot@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.

New file creation include source path in package name

3 participants