Skip to content

Conversation

@RicardoASJunior
Copy link
Collaborator

Refactored code to improve organization:

  • Request sending function moved to api/ccs/
  • Endpoint structure adjusted to commands/ccs/

@RicardoASJunior RicardoASJunior merged commit 56d60d8 into prerelease Sep 26, 2025
3 checks passed
@LeoAnders LeoAnders added the invalid This doesn't seem right label Sep 26, 2025
@LeoAnders
Copy link
Collaborator

This pull request was created from an outdated branch and ended up overwriting newer commits on prerelease.

A corrected pull request has been opened here: #10.

LeoAnders pushed a commit that referenced this pull request Oct 7, 2025
* chore (api): restructure to use request util

* docs: add changelog entry for API refactor

* style: fix prettier formatting issues
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

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants