-
Notifications
You must be signed in to change notification settings - Fork 409
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.
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
fix: fix MD labels in org browser #5996
Merged
Merged
Conversation
This file contains 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
CristiCanizales
approved these changes
Jan 7, 2025
mingxuanzhangsfdx
added a commit
that referenced
this pull request
Jan 15, 2025
* docs: update README and developing.md (#5960) * Update README.md * Update README.md * Update developing.md * chore(deps): bump cross-spawn from 7.0.3 to 7.0.5 in /packages/salesforcedx-utils-vscode (#5949) * chore(deps): bump cross-spawn in /packages/salesforcedx-utils-vscode Bumps [cross-spawn](/~https://github.com/moxystudio/node-cross-spawn) from 7.0.3 to 7.0.5. - [Changelog](/~https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.5) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> * chore: bump cross-spawn to v7.0.6 and @types/cross-spawn to v6.0.6 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * chore: update to version 62.8.0 * fix: update some md labels for org browser (#5967) * feat: add missing labels for org browser * chore: labels missing in japanese file * chore: update to version 62.9.0 * Mz/disable apex ls error telemetry (#5969) * feat: add an user setting to disable error to telemetry by default * chore: fix linting * chore: updated jar * chore: update to version 62.9.1 * chore: generated CHANGELOG for release/v62.9.1 * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * chore: updated SHA256 v62.9.1 [skip ci] * chore: update to version 62.10.0 * fix: prevent `Request textDocument/documentHighlight failed with message: Debug Failure. False expression. Code: -32603` error when selecting Javascript code in the embedded `<script>` tag (#5975) * Visualforce extension - prevent "Request textDocument/documentHighlight failed with message: Debug Failure. False expression. Code: -32603" error when selecting javascript code in the <script> tag embedded in Visualforce page/component (#5279) * Update packages/salesforcedx-visualforce-language-server/src/modes/javascriptMode.ts --------- Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> * chore: update to version 62.11.0 * chore: update to version 62.12.0 * chore: generated CHANGELOG for release/v62.12.0 * fix: use correct telemetry enable setting (#5986) * fix: use correct telemetry enable setting * fix: empty commit to trigger salesforce cla * fix: empty commit to trigger salesforce cla * fix: empty commit to trigger salesforce cla * fix: renaming variables to trigger salesforce cla * fix: updating the default value to all --------- Co-authored-by: Madhur Shrivastava <madhur.shrivastava@madhurs-ltmzugg.internal.salesforce.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> * Update CHANGELOG.md * Update CHANGELOG.md Sonal's edits. * chore: updated SHA256 v62.12.0 [skip ci] * chore(deps): bump nanoid from 3.3.6 to 3.3.8 (#5973) Bumps [nanoid](/~https://github.com/ai/nanoid) from 3.3.6 to 3.3.8. - [Release notes](/~https://github.com/ai/nanoid/releases) - [Changelog](/~https://github.com/ai/nanoid/blob/main/CHANGELOG.md) - [Commits](ai/nanoid@3.3.6...3.3.8) --- updated-dependencies: - dependency-name: nanoid dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * chore: update to version 62.13.0 * chore: generated CHANGELOG for release/v62.13.0 * fix: fix MD labels in org browser (#5996) * chore: update changelog * chore: updated SHA256 v62.13.0 [skip ci] * fix: excluding schema from bundling to avoid warning (#6000) * fix: excluding schema from bundling to avoid warning * fix: excluding schema from bundling to avoid warning * fix: hide commands outside of forceapp main default (#6003) * fix: hide commands outside of forceapp main default * chore: hide in context menu from explorer view * chore: update to version 62.14.0 * Revert "fix: hide commands outside of forceapp main default (#6003)" (#6004) This reverts commit 7464a58. Co-authored-by: peternhale <peternhale@users.noreply.github.com> * chore: update to version 62.14.1 * chore: generated CHANGELOG for release/v62.14.1 * Update CHANGELOG.md * chore: updated SHA256 v62.14.1 [skip ci] * fix: resolve call (#6006) * chore: dummy * fix: parameter * fix: parameter * chore: revert jorje jar --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> Co-authored-by: Release Bot <svc_idee_bot@salesforce.com> Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> Co-authored-by: AndrewStopchenko-SO <63401193+AndrewStopchenko-SO@users.noreply.github.com> Co-authored-by: madhur310 <madhur.shrivastava@salesforce.com> Co-authored-by: Madhur Shrivastava <madhur.shrivastava@madhurs-ltmzugg.internal.salesforce.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> Co-authored-by: Sonal Budhiraja <57969266+sbudhirajadoc@users.noreply.github.com>
daphne-sfdc
added a commit
that referenced
this pull request
Feb 28, 2025
* feat: create apex action from selected method/this class (#5950) * feat: create apex action from selected method * chore: refactor * feat: create apex action from this class * chore: messaging and when clause * chore: refactor, comments and tests * chore: revert changes * chore: add sourceUri param --------- Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> * chore: e2e tests for apex actions (#5956) * chore: e2e tests for apex actions * chore: className * chore: tests fix? * feat: metadata eligibility (#5958) * feat: logic for validating eligibility * feat: configurations of the definitions of eligible apex classes/methods/properties * chore: lint * test: validateEligibility * chore: apply reviewers suggestions * test: requestTarget * test: metadata eligibility * fix: build * Update packages/salesforcedx-vscode-apex/src/messages/i18n.ts Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * Update packages/salesforcedx-vscode-apex/src/messages/i18n.ts Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * Update packages/salesforcedx-vscode-apex/src/messages/i18n.ts Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * chore: combine withsharing * Update packages/salesforcedx-vscode-apex/src/commands/metadataOrchestrator.ts Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * Update packages/salesforcedx-vscode-apex/src/commands/metadataOrchestrator.ts Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * Update packages/salesforcedx-vscode-apex/src/commands/metadataOrchestrator.ts Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * chore: update with latest jorje jar --------- Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * feat: shell for deploy and retrieve oas (#5962) * feat: shell for deploy and retrieve commands for OAS * chore: lint * chore: messages * feat: implement metadata extraction (#5961) * feat: logic for validating eligibility * feat: configurations of the definitions of eligible apex classes/methods/properties * chore: lint * test: validateEligibility * chore: apply reviewers suggestions * test: requestTarget * test: metadata eligibility * chore: replace extraction * chore: working * chore: lint * chore: update jar * chore: lint error * chore: tests, labels, nits and retries for llm * chore: nit * fix: no active editor bug when right clicking in project explorer * chore: pr comments * chore: test * chore: any to type * chore: messages in japanese file --------- Co-authored-by: mingxuanzhang <mingxuanzhang@salesforce.com> * build: merge develop to oas (#5968) * docs: update README and developing.md (#5960) * Update README.md * Update README.md * Update developing.md * chore(deps): bump cross-spawn from 7.0.3 to 7.0.5 in /packages/salesforcedx-utils-vscode (#5949) * chore(deps): bump cross-spawn in /packages/salesforcedx-utils-vscode Bumps [cross-spawn](/~https://github.com/moxystudio/node-cross-spawn) from 7.0.3 to 7.0.5. - [Changelog](/~https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.5) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> * chore: bump cross-spawn to v7.0.6 and @types/cross-spawn to v6.0.6 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * chore: update to version 62.8.0 * build: merge develop to apex-oas --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> Co-authored-by: Release Bot <svc_idee_bot@salesforce.com> * Mz/distinct eligibility (#5965) * feat: add distinct configs for apex action and general oas eligibility * chore: baseline fallback * chore: revert changes * fix: adjust for new eligibility properties (#5972) @W-17249057@ * feat: hide deploy and retrieve oas commands (#5978) * chore: eligibility criteria for class and method annotations (#5976) * Revert "chore: eligibility criteria for class and method annotations (#5976)" (#5980) This reverts commit f5f04e6. * chore: eligibility criteria for class and method annotations (#5981) * chore: eligibility criteria for class and method annotations * chore: update rest-related annotations * feat: latest jorje jar * chore: pin vsp version * feat: gather context (#5988) * feat: gather context * chore: update jar * chore: eligibility criteria for class and method annotations (#5981) * chore: eligibility criteria for class and method annotations * chore: update rest-related annotations * feat: latest jorje jar * chore: pin vsp version * feat: gather context * chore: update jar * chore: reorder requests * chore: reorder imports * chore: renaming after pr comments --------- Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> * chore: merge develop into feat/apex-oas (#5995) * Merge develop into feat/apex-oas * feat: use new JAR that contains disable Apex error telemetry changes * fix: fix MD labels in org browser * feat: create/update an ESR record with generated OAS spec (#5993) * feat: esr creation * chore: lint * chore: pr comments * chore: use sdr to get default folder for esr * chore: sort imports * chore: pr comments * chore: use fast-xml-parser * feat: use new llm service (#6002) * feat: use new llm service * chore: remove commented lines * chore: fixup prompt to remove explanation * fix: change instance name from salesforcedx-vscode to salesforcedx-vscode-apex --------- Co-authored-by: peternhale <peter.hale@salesforce.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * feat: exclude auraenabled as valid annotation (#6005) * chore: merge develop into feat/apex-oas (#6008) * docs: update README and developing.md (#5960) * Update README.md * Update README.md * Update developing.md * chore(deps): bump cross-spawn from 7.0.3 to 7.0.5 in /packages/salesforcedx-utils-vscode (#5949) * chore(deps): bump cross-spawn in /packages/salesforcedx-utils-vscode Bumps [cross-spawn](/~https://github.com/moxystudio/node-cross-spawn) from 7.0.3 to 7.0.5. - [Changelog](/~https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.5) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> * chore: bump cross-spawn to v7.0.6 and @types/cross-spawn to v6.0.6 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * chore: update to version 62.8.0 * fix: update some md labels for org browser (#5967) * feat: add missing labels for org browser * chore: labels missing in japanese file * chore: update to version 62.9.0 * Mz/disable apex ls error telemetry (#5969) * feat: add an user setting to disable error to telemetry by default * chore: fix linting * chore: updated jar * chore: update to version 62.9.1 * chore: generated CHANGELOG for release/v62.9.1 * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * chore: updated SHA256 v62.9.1 [skip ci] * chore: update to version 62.10.0 * fix: prevent `Request textDocument/documentHighlight failed with message: Debug Failure. False expression. Code: -32603` error when selecting Javascript code in the embedded `<script>` tag (#5975) * Visualforce extension - prevent "Request textDocument/documentHighlight failed with message: Debug Failure. False expression. Code: -32603" error when selecting javascript code in the <script> tag embedded in Visualforce page/component (#5279) * Update packages/salesforcedx-visualforce-language-server/src/modes/javascriptMode.ts --------- Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> * chore: update to version 62.11.0 * chore: update to version 62.12.0 * chore: generated CHANGELOG for release/v62.12.0 * fix: use correct telemetry enable setting (#5986) * fix: use correct telemetry enable setting * fix: empty commit to trigger salesforce cla * fix: empty commit to trigger salesforce cla * fix: empty commit to trigger salesforce cla * fix: renaming variables to trigger salesforce cla * fix: updating the default value to all --------- Co-authored-by: Madhur Shrivastava <madhur.shrivastava@madhurs-ltmzugg.internal.salesforce.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> * Update CHANGELOG.md * Update CHANGELOG.md Sonal's edits. * chore: updated SHA256 v62.12.0 [skip ci] * chore(deps): bump nanoid from 3.3.6 to 3.3.8 (#5973) Bumps [nanoid](/~https://github.com/ai/nanoid) from 3.3.6 to 3.3.8. - [Release notes](/~https://github.com/ai/nanoid/releases) - [Changelog](/~https://github.com/ai/nanoid/blob/main/CHANGELOG.md) - [Commits](ai/nanoid@3.3.6...3.3.8) --- updated-dependencies: - dependency-name: nanoid dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> * chore: update to version 62.13.0 * chore: generated CHANGELOG for release/v62.13.0 * fix: fix MD labels in org browser (#5996) * chore: update changelog * chore: updated SHA256 v62.13.0 [skip ci] * fix: excluding schema from bundling to avoid warning (#6000) * fix: excluding schema from bundling to avoid warning * fix: excluding schema from bundling to avoid warning * fix: hide commands outside of forceapp main default (#6003) * fix: hide commands outside of forceapp main default * chore: hide in context menu from explorer view * chore: update to version 62.14.0 * Revert "fix: hide commands outside of forceapp main default (#6003)" (#6004) This reverts commit 7464a58. Co-authored-by: peternhale <peternhale@users.noreply.github.com> * chore: update to version 62.14.1 * chore: generated CHANGELOG for release/v62.14.1 * Update CHANGELOG.md * chore: updated SHA256 v62.14.1 [skip ci] * fix: resolve call (#6006) * chore: dummy * fix: parameter * fix: parameter * chore: revert jorje jar --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> Co-authored-by: Release Bot <svc_idee_bot@salesforce.com> Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> Co-authored-by: AndrewStopchenko-SO <63401193+AndrewStopchenko-SO@users.noreply.github.com> Co-authored-by: madhur310 <madhur.shrivastava@salesforce.com> Co-authored-by: Madhur Shrivastava <madhur.shrivastava@madhurs-ltmzugg.internal.salesforce.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> Co-authored-by: Sonal Budhiraja <57969266+sbudhirajadoc@users.noreply.github.com> * feat: support ESR changes for 254 (#6009) * feat: esr modifications for 254 * chore: sort imports * chore: refine structure * chore: lint * chore: catalogedApiVersion null * chore: missing label * chore: use max api version from connection * feat: add operations tag to esr xml and operationId to yaml (#6013) * feat: add operations tag to esr xml and operationId to yaml * chore: lint * chore: update operations on overwrite * chore: ? * fix: make esr deployable with operations (#6014) * feat: prompt generation strategy (#6010) * feat: prompt generation strategy * chore: reduce dummy * chore: rename class * feat: refactor generation strategy bidding * chore: add test in include option in tsconfig * Update packages/salesforcedx-vscode-apex/src/oas/generationStrategy/methodByMethodStrategy.ts Co-authored-by: peternhale <peternhale@users.noreply.github.com> * Update packages/salesforcedx-vscode-apex/src/oas/promptGenerationOrchestrator.ts Co-authored-by: peternhale <peternhale@users.noreply.github.com> * chore: remove useless comments * chore: apply reviewer suggestion * Update packages/salesforcedx-vscode-apex/src/oas/generationStrategy/methodByMethodStrategy.ts Co-authored-by: peternhale <peternhale@users.noreply.github.com> * chore: update after committing strategy * chore: add comments --------- Co-authored-by: peternhale <peternhale@users.noreply.github.com> * chore: fix typo in apex_class_not_valid message (#6020) * feat: update jar with changes in lsp to add comments in context (#6025) * feat: update jar with changes in lsp to add comments in context * feat: update jar with changes in lsp to add comments in context * fix: add comment property to context (#6026) @W-17645605@ moved comments from ApexClassOASGatherContextResponse and added comment proeprty to class/method/property * chore: remove deploy and retrieve commands for oas and hide create oas from method command (#6029) * Madhur/w 17619393 (#6017) * feat: adding spectral library and validation based on OAS3 spec * feat: adding spectral library and validation based on OAS3 spec * feat: adding spectral library and validation based on OAS3 spec * feat: adding spectral library and validation based on OAS3 spec * fix: increase minimum supported node version to v20.17.0 * feat: adding spectral library and validation based on OAS3 spec --------- Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * feat: add option to manually merge new ESR with existing ESR (#6027) * fix: use ClassName.externalServiceRegistration-meta.xml for ESR name for both class and method * feat: add new button to manually merge ESR instead of overwrite if ESR already exists * feat: create new ESR with className_currentTimestamp if ESR already exists and user chooses merge * feat: popup diff window comparing original and new ESR files for manual merge * fix: use path.basename instead of filename.split * feat: different success message for merge and overwrite cases * fix: rename saveOasAsErsMetadata() to saveOasAsEsrMetadata() * fix: add .github/actions/*/lib/src/*.js to .gitignore * fix: handle the case where the user overwrites the existing ESR with a different named credential * fix: remove duplicate Cancel button in popup modal * fix: handle the case where the user does not provide a named credential or provides only whitespace * chore: disable sf.create.apex.action.method in editor view context menu * fix: update file_exists message * fix: move timestamped ESR files to esr_files_for_merge folder in root directory of workspace * fix: do not change class name inside file contents in the new ESR for merge * chore: remove duplicate cleanupYaml() function * feat: bump SDR to v12.14.0, STL to v7.3.0, Core to v8.8.2 (#6035) * W-17681241 - feat: merge generation strategy into feature branch (#6034) * Mz/apply generation strategy (#6018) * feat: integrate apex action controller with prompt generation orchestrator * chore: update framework * chore: add error entries * feat: whole class gen strategy (#6024) * feat: integrate apex action controller with prompt generation orchestrator * feat: whole class gen strategy * chore: sort imports * chore: asdfghj --------- Co-authored-by: mingxuanzhang <mingxuanzhang@salesforce.com> * feat: update jar with changes in lsp to add comments in context (#6028) * feat: update jar with changes in lsp to add comments in context (#6025) * feat: update jar with changes in lsp to add comments in context * feat: update jar with changes in lsp to add comments in context * fix: add comment property to context (#6026) @W-17645605@ moved comments from ApexClassOASGatherContextResponse and added comment proeprty to class/method/property --------- Co-authored-by: peternhale <peternhale@users.noreply.github.com> * feat: method-by-method OAS doc generation strategy (#6023) * feat: create open api doc method by method * chore: update for generating ESR * chore: refine * chore: refine prompt structure * feat: enable comments for prompt generation * chore: sync jorje jar * chore: sync generation strategy with latest change (#6031) * feat: update jar with changes in lsp to add comments in context (#6025) * feat: update jar with changes in lsp to add comments in context * feat: update jar with changes in lsp to add comments in context * fix: add comment property to context (#6026) @W-17645605@ moved comments from ApexClassOASGatherContextResponse and added comment proeprty to class/method/property * chore: remove deploy and retrieve commands for oas and hide create oas from method command (#6029) * Madhur/w 17619393 (#6017) * feat: adding spectral library and validation based on OAS3 spec * feat: adding spectral library and validation based on OAS3 spec * feat: adding spectral library and validation based on OAS3 spec * feat: adding spectral library and validation based on OAS3 spec * fix: increase minimum supported node version to v20.17.0 * feat: adding spectral library and validation based on OAS3 spec --------- Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> --------- Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> Co-authored-by: madhur310 <madhur.shrivastava@salesforce.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * chore: remove eslint disable * chore: move schemas to oas folder * chore: sort imports --------- Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> Co-authored-by: mingxuanzhang <mingxuanzhang@salesforce.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> Co-authored-by: madhur310 <madhur.shrivastava@salesforce.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * W-17654307 - chore: improve language in texts (#6043) * chore: improve language in texts * chore: update test * feat: put OAS doc in separate YAML file if sfdx-project.json contains "decomposeExternalServiceRegistrationBeta" (#6041) * feat: do not put schemas in ESR XML if sfdx-project.json contains decomposeExternalServiceRegistrationBeta * feat: generate separate YAML file containing OAS doc contents * feat: add diff for YAML files of ESR * fix: YAML OAS docs can use ' instead of ' * chore: add comments * fix: replace console.error() with throw new Error * fix: handle overwrite case for decomposed ESR files by overwriting the YAML file * fix: open both XML and YAML files of the ESR after generation if the ESR is decomposed * fix: handle the case for overwrite where the ESR is decomposed but the <schemas> element still exists in the ESR XML * chore: cleanup * fix: remove unnecessary check for the presence of sfdx-project.json * fix: rewrite isESRDecomposed() to use getSfProjectJson() from @salesforce/core-bundle * fix: handle the case for overwrite where the ESR is not decomposed but the <schemas> element is not present in the ESR XML * refactor: move creating the contents of the ESR XML to createESRObject() helper function * chore: add comments * refactor: move replacing the ESR XML file ending with the YAML file ending to a helper function * refactor: move opening the diff editor containing the two ESR files to a helper function * fix: only call isESRDecomposed() once at the beginning of createApexAction() * refactor: make isESRDecomposed a field in the ApexActionController class * Update packages/salesforcedx-vscode-apex/src/commands/apexActionController.ts Co-authored-by: peternhale <peternhale@users.noreply.github.com> * fix: void this.openDiffFile() * fix: update index.ts and createApexAction.ts so that apexActionController calls the initialize() function which contains isESRDecomposed --------- Co-authored-by: peternhale <peternhale@users.noreply.github.com> * W-17656525 - W-17684256 - feat: show validations in problems tab and add mandatory op ids (#6037) * feat: show validations in problems tab and add mandatory op ids * chore: remove operationId logic from methodbymethod strategy * chore: doc is an object and not a string refactor * Madhur/w 17619422 (#6042) * feat: adding custom salesforce specific rulesets for oas doc * feat: adding custom salesforce specific rulesets for oas doc * feat: adding custom salesforce specific rulesets for oas doc * Cristi/prompts json (#6045) * feat: move prompts to json in .sfdx folder * chore: sort imports * chore: diagnosticCollection * chore: use yaml instead of json * W 17693649 (#6047) * feat: adding rules for operations * feat: adding rules for requestBody * feat: adding rules for parameters * feat: adding rules for responses and media types * feat: method validation step in OAS (#6038) * feat: method validation step * feat: display errors in problem tab from method validation * fix: test * chore: set default formatter in vscode and revert format changes * Update packages/salesforcedx-vscode-apex/src/oas/documentProcessorPipeline/oasProcessor.ts Co-authored-by: peternhale <peternhale@users.noreply.github.com> * chore: apply suggestions * chore: fix merging errors * chore: refactor errors in processor * chore: fix --------- Co-authored-by: peternhale <peternhale@users.noreply.github.com> * fix: exclude org perm protected properties in esr (#6048) @W-17712234@ remove org perm protected properties * W-17695231 - chore: update prompts to ensure compliance with validation rules (#6050) * chore: update prompts to ensure compliance with validation rules * chore: update prompts * feat: call Mulesoft ACD extension with command "mule-dx-api.open-api-project" and OAS document vscode URI when new OAS doc created (#6049) * feat: call Mulesoft ACD extension with command "mule-dx-api.open-api-project" and OAS document vscode URI when new OAS doc created * fix: address code review comments * feat: block oas until a4d is activated (#6046) * feat: register command to notify a4d activated status * feat: register command for a4d deactivated notifying * feat: use a4d isenabled context value * chore: remove usages * fix: updating ruleset to find parameter related issue if parameter is defined and tests (#6051) * fix: updating ruleset to find parameter related issue if parameter is defined and tests * fix: disable info-contact & operation-tag-defined rule * feat: update jorje with namespace check (#6056) * fix: exclude jsonc-parser from apex bundling (#6057) @W-17751384@ exclude jsonc-parser from apex ext bundle * chore: stick to method-by-method strategy (#6059) * fix: parameters missing for http get * chore: stick to method by method strategy * chore: add beta tag to command label (#6065) * [W-17756721] fix: add annotation params to prompts (#6063) * fix: add annotation params to prompts @W-17756721@ add annotation params from context to prompts add process step to resolve semantically duplicate paths * Update packages/salesforcedx-utils-vscode/src/helpers/utils.ts Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> * chore: apply review suggestions modified annotation prompt builder to exclude Parameters text when no params --------- Co-authored-by: mingxuanzhang <mingxuanzhang@salesforce.com> Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> * fix: correct to services/apexrest (#6066) * fix: remove duplicate rule (#6073) * feat: add logic to rerun validations and show validations in correct file (#6070) * feat: determine operationId and request path (#6071) * feat: determine request path and operationId * chore: required parameters are always before optional ones * chore: use jsonpath-plus rather than raw operations * fix: make 'without sharing' and 'inherited sharing' eligible for OAS generation (#6080) * fix: do not show errors in Problems tab when Mulesoft extension is installed (#6077) * fix: do not show errors in Problems tab when Mulesoft extension is installed * fix: move check for Mulesoft extension to oasUtils.ts * fix: apply suggestion from code review * fix: log generation strategy stream (#6085) @W-17764181 Add local capture of generation strategy stream Add internal config for strategy selection * chore: bump service provider to consume callLLMOptions (#6091) * chore: node 20.17.0 rather than 20.17 * chore: use original order in package.nls.json * fix: add apex_oas_generation_strategy to package.nls.ja.json * fix: add apex_oas_general_method_access-modifiers and apex_oas_general_property_access-modifiers to package.nls.ja.json * chore: remove unused deploy_oas_text and retrieve_oas_text entries * chore: new package-lock.json * fix: bump jsonpath-plus to v10.3.0 to resolve a Synk security issue * [W-17812942] fix: use correct id for mulesoft extension (#6107) * [W-17830514] fix: adjust prompts and handlers for json response (#6095) * refactor: wip * chore: apply review comments * chore: wip * chore: wip * chore: wip * chore: wip * fix: rename to generationInteractionLogger.ts * fix: add apex_oas_generation_strategy and apex_oas_generation_schema to package.nls.json * chore: remove unused import * fix: update system prompt to inline schemas instead of producing a #/components/schemas entry * chore: wip * chore: retries * chore: remove mandatory components section,non-2xx responses,non-related request methods (#6108) * fix: [W-17881383] add descriptions when missing @W-17881383@ Added corrective actions to ensure descriptions appear where required. * chore: add class name and comment to prompt --------- Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> * feat: disabling operation-tags rule (#6109) * [W-17847229] fix: add telemetry for oas generations (#6110) * chore: wip * chore: wip * fix: add telemetry for oas generations @W-17847229@ Add telemetry for OAS generation * chore: adjust telemetry event name for oas change event name to be similar to the IDE command being run * chore: fix typo * chore: fix typos * chore: address review concerns * Apply suggestions from code review Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> * chore: fix description correction bug * chore: formatting --------- Co-authored-by: Daphne Yang <daphne.yang@salesforce.com> Co-authored-by: Daphne Yang <139700604+daphne-sfdc@users.noreply.github.com> * chore: resolve conflicts (#6113) * W-17146736 test: generation strategy (#6112) * chore: extract utils from class * test: formatUtils * test: buildPromptUtils * chore: fix * chore: remove unused deps * test: promptHandler * chore: adjust reference for win * [W-17920354] fix: windows bug for handling path (#6116) * fix: properly handle uri transforms on endpoints @W-17920354@ Add custom Apex Language Client methods for eligibility and context * chore: fix unit test * fix: llm logging fails on windows (#6117) * chore: update jar with fix for windows * chore: fix merge filename to remove colon --------- Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> Co-authored-by: Cristina Isabel Cañizales <cristina.canizales@salesforce.com> * fix: url in windows in problems tab (#6118) * fix: problems tab shows problems for newly generated OAS doc with timestamp in manual merge case (#6119) --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Cristina Cañizales <113132642+CristiCanizales@users.noreply.github.com> Co-authored-by: Mingxuan Zhang <132491513+mingxuanzhangsfdx@users.noreply.github.com> Co-authored-by: Cristina Isabel Cañizales <cristina.canizales@salesforce.com> Co-authored-by: mingxuanzhang <mingxuanzhang@salesforce.com> Co-authored-by: peternhale <peternhale@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Release Bot <svc_idee_bot@salesforce.com> Co-authored-by: peternhale <peter.hale@salesforce.com> Co-authored-by: madhur310 <madhur.shrivastava@salesforce.com> Co-authored-by: AndrewStopchenko-SO <63401193+AndrewStopchenko-SO@users.noreply.github.com> Co-authored-by: Madhur Shrivastava <madhur.shrivastava@madhurs-ltmzugg.internal.salesforce.com> Co-authored-by: Sonal Budhiraja <57969266+sbudhirajadoc@users.noreply.github.com>
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.
[skip-validate-pr]