Skip to content

Fixed: Intellisense Hover Description RegExp pattern#248

Open
donnyh13 wants to merge 1 commit into
loganch:masterfrom
donnyh13:hover-description
Open

Fixed: Intellisense Hover Description RegExp pattern#248
donnyh13 wants to merge 1 commit into
loganch:masterfrom
donnyh13:hover-description

Conversation

@donnyh13
Copy link
Copy Markdown

@donnyh13 donnyh13 commented May 11, 2026

Description

🔗 Linked GitHub Issues

#247

Closes #247

📋 What is the current behavior?

Currently UDF function header descriptions are not recognized for Intellisense, showing "undefined" rather than the description. This was introduced when modifying utils.js.

`(?:\\r\\n?\\n?\\s*;\\s+Description\\s*\\.+:\\s+(?<description>.+))?` +

🚀 What is the new behavior?

The RegExp Pattern had a small incorrect pattern and needed a slight change, and it now (seems) to work correctly.


Type of changes

  • 🪲 Bugfix (change which fixes an issue)
  • ⭐ New Feature (change which adds functionality)
  • 🔒 Security fix (change which improves security)
  • 🔮 Code style update (formatting, renaming)
  • 🔨 Refactoring (code optimization without functional change)
  • 📚 Documentation (updates to README or docs)
  • ⚙️ Build or CI related changes
  • 🧿 Other type

Breaking changes 🔥

  • Yes
  • No

How and where was this tested?

🖥️ Describe where you tested your changes

System:

  • Windows 11 (x64)
  • Windows 10 (x64)
  • Windows 10 (x86)
  • other Windows version
  • other operating system

Context:

  • in Browser <name and version>
  • with Go <version>
  • with Node.js <version>
  • with AutoIt <version>
  • with .NET (C#) <version>
  • with ...

🔬 Describe how you tested your changes

  • Manually tested in system and context shown above
  • Ran automatic tests (unit tests, integration tests, etc.)
  • Other ways

Checklist

  • I have read and understood the available contributing guidelines.
  • I have ensured my code follows the available code conventions.
  • I have reviewed my changes.
  • I have made corresponding changes to the documentation (if applicable).
  • I have added/updated necessary tests to cover the changes (if applicable).
  • I have checked for potential security implications.

Additional context

Screenshots

Note to reviewers

@RommelSanchez
Copy link
Copy Markdown

It works! Tnx!

@donnyh13
Copy link
Copy Markdown
Author

Thanks for checking and reporting back @RommelSanchez, It's much appreciated.

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.

[BUG]: Regression in v1.5.0: IntelliSense shows "undefined" for documentation of script functions

2 participants