Skip to content

Add experimental funcDefSearch feature #623

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 32 commits into from
Mar 14, 2025
Merged

Add experimental funcDefSearch feature #623

merged 32 commits into from
Mar 14, 2025

Conversation

mark-wiemer
Copy link
Owner

  • Recursively resolve included files (with cycle detection)
  • Prioritize included files over global matches
  • Match local library files with exact name
  • Match local library files with prefix name

There is still no feature that searches based on user or standard libraries.

Known issues documented in settings.md

@mark-wiemer
Fixup showOutput Chinese description
5e09a8b
@mark-wiemer
Remove old todo
c6a48b8
@mark-wiemer
Add experimental v1IncludeTree setting (unused)
a3dc0b3
@mark-wiemer
Cleanup Parser code a bit
97aec1f
@mark-wiemer
Cleanup getPathsToBuild
43cd51f
@mark-wiemer
Cleanup defProvider.utils.ts
30c1e64
@mark-wiemer
Move #include utils to common folder
6a1ada0
@mark-wiemer
Document v1IncludeTree
35c7e87
@mark-wiemer
Cache and prioritize included scripts
05f8fe9
@mark-wiemer
Document pathsToBuild
53d42dd
@mark-wiemer
Cleanup parser e2e tests
0937bb2
@mark-wiemer
Handle included file not existing
c8cb56b
@mark-wiemer
Test for nonexistent included file
ba50cc2
@mark-wiemer
Update changelog
fe65cf2
@mark-wiemer
Handle thisScript not being part of localCache
74a317d
@mark-wiemer
Revert "Handle thisScript not being part of localCache"
00eccef 
This reverts commit 74a317d.
@mark-wiemer
Cleanup includeTests
e15b243
@mark-wiemer
Move getFuncDefByName to utils file for unit tests
0aaae6c
@mark-wiemer
Add validate:test:e2e to pick up from e2e test phase
955b9b8
@mark-wiemer
Add deep search
9e9eb3a
@mark-wiemer
Clarify settings.md
5b77691
@mark-wiemer
Cleanup settings.md
09e7f42
@mark-wiemer
First local lib test
fb665b8
@mark-wiemer
More tests
cbd455d
@mark-wiemer
Reduce test boilerplate
a60c079
@mark-wiemer
Cleanup tests
005b11d
@mark-wiemer
Add test for prefix search
e50a097
@mark-wiemer
Finalize v1IncludeTree
31cfdfb
@mark-wiemer
Document known issue, remove suite.only
ffcca6c
@mark-wiemer
Rename v1IncludeTree to funcDefSearch
e106c7e
@mark-wiemer mark-wiemer enabled auto-merge (squash) March 14, 2025 03:15
@mark-wiemer mark-wiemer merged commit 76b90fa into main Mar 14, 2025
3 checks passed
@mark-wiemer mark-wiemer deleted the include-tree branch March 14, 2025 03:18
@mark-wiemer mark-wiemer mentioned this pull request Mar 21, 2025
Merged
@mark-wiemer mark-wiemer mentioned this pull request Apr 6, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
AHK++
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mark-wiemer