Unify Go to Definition(F12) and Ctrl+Click through CCS API resolution #20
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nding - Introduced DefinitionResolverClient for REST API resolution - Implemented definitionLookup feature (extractQuery + lookup) for robust query handling - Added `PrioritizedDefinitionProvider` to prefer CCS resolver before fallback - Implemented new command vscode-objectscript.ccs.goToDefinition with API-first fallback - Updated package.json to bind F12 and menus to the new command for ObjectScript files - Registered new command in extension.ts and integrated telemetry - New goToDefinitionLocalFirst command integrates CCS API before native definition
…mentLinkProvider`
LeoAnders
added a commit
that referenced
this pull request
Oct 7, 2025
…#20) * revert: remove cross-workspace definition lookup (45b7b82) * feat: add API-based Go to Definition command and integrate with keybinding - Introduced DefinitionResolverClient for REST API resolution - Implemented definitionLookup feature (extractQuery + lookup) for robust query handling - Added `PrioritizedDefinitionProvider` to prefer CCS resolver before fallback - Implemented new command vscode-objectscript.ccs.goToDefinition with API-first fallback - Updated package.json to bind F12 and menus to the new command for ObjectScript files - Registered new command in extension.ts and integrated telemetry - New goToDefinitionLocalFirst command integrates CCS API before native definition * feat: support cross-namespace definition lookup and request logging * feat: implement full Ctrl+Click support with `CCS API` resolution * feat: remove visual underline from `DocumentLinks` in `DefinitionDocumentLinkProvider`
LeoAnders
added a commit
that referenced
this pull request
Oct 7, 2025
* feat: enable cross-workspace definition lookup * feat: add ObjectScript enter rules for semicolon continuation * feat: auto-indent dot syntax on enter for `.mac` / `.int` routines * feat: add resolveContextExpression command `Ctrl+Alt+Space` to call API and insert returned code * fix: error Insert `enter` prettier/prettier * Chore API restructure to use request util (#9) * chore (api): restructure to use request util * docs: add changelog entry for API refactor * style: fix prettier formatting issues * ci: add `target_commitish` to correctly reference commit in prerelease tags * refactor: reorganize CCS module layout * feat: add `core` and centralized `config` scaffolds * feat: add ContextExpressionClient and centralize route handling - Introduced ContextExpressionClient to encapsulate API calls for \/resolveContextExpression\, applying settings, timeout and logging - Updated contextHelp.ts to consume the new client instead of creating SourceControlApi manually - Added routes.ts with BASE_PATH and resolveContextExpression route - Updated extension.ts to import CCS commands/providers from barrel - Adapted SourceControlApi client to use centralized routes (only resolveContextExpression for now) * feat: reorganize SourceControl API with dedicated clients folder * Fix prevent unwanted semicolon insertion (#13) Fix unwanted semicolon insertion in ObjectScript dot-prefixed blocks: auto-continuation now inserts the copied prefix only when the target line is empty and preserves ; only if it existed on the previous line, and a regression test ensures moving lines across dot-prefixed semicolon comments does not inject semicolons. * Feat add Ctrl+Q to fetch global documentation from selection and print to Output (#17) Add Ctrl+Q shortcut to fetch global documentation for the current selection/line and print the response to the Output. * Unify Go to Definition(F12) and Ctrl+Click through CCS API resolution (#20) * revert: remove cross-workspace definition lookup (45b7b82) * feat: add API-based Go to Definition command and integrate with keybinding - Introduced DefinitionResolverClient for REST API resolution - Implemented definitionLookup feature (extractQuery + lookup) for robust query handling - Added `PrioritizedDefinitionProvider` to prefer CCS resolver before fallback - Implemented new command vscode-objectscript.ccs.goToDefinition with API-first fallback - Updated package.json to bind F12 and menus to the new command for ObjectScript files - Registered new command in extension.ts and integrated telemetry - New goToDefinitionLocalFirst command integrates CCS API before native definition * feat: support cross-namespace definition lookup and request logging * feat: implement full Ctrl+Click support with `CCS API` resolution * feat: remove visual underline from `DocumentLinks` in `DefinitionDocumentLinkProvider` * chore: add dedicated CHANGELOG.md for internal Consistem updates --------- Co-authored-by: Ricardo Alves <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DocumentLinksfor cleaner inline display