Skip to content

Enhance Output Options for Grok Responses & add devcontainer #2

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kloodz-mrioux
Copy link

Grok Response 1

Here's a pull request (PR) message based on the diff you've provided:


Title: Enhance Output Options for Grok Responses

Description:

This PR introduces enhancements to the output display options for Grok responses in the VSCode extension:

  • New Output Method: Added a new configuration option tabPreview which allows users to view Grok responses in a markdown preview tab. This can be selected from the configuration settings under outputMethod.

    "enum": [
      "tab",
      "tabPreview",
      "outputChannel"
    ]
  • Implementation of tabPreview:

    • Created a new function displayResponseInTabPreview in src/display.ts to handle the new output method. This function:
      • Opens each Grok response in a separate markdown preview tab.
      • Provides a user-friendly message once all responses are displayed.
  • Configuration and Constants:

    • Updated src/const.ts to include OUTPUT_METHOD_TAB_PREVIEW.
    • Removed activationEvents from package.json as they are no longer needed or have been moved elsewhere.
  • Dependencies:

    • Updated esbuild from version 0.24.2 to 0.25.1 for better build performance and compatibility.
  • TypeScript Configuration:

    • Added "skipLibCheck": true to tsconfig.json to skip type checking of all declaration files (*.d.ts), which can speed up compilation times.

Changes:

  • Removed unused activationEvents from package.json.
  • Updated dependencies in package.json.
  • Added new constants and functions to support the tabPreview feature.

Testing:

  • The tabPreview functionality needs further testing, especially regarding the closing of documents without prompting to save. This part is currently commented out due to potential issues.

Please review the changes, and let me know if there are any concerns or if further adjustments are needed before merging.


This PR message outlines the changes, explains the new features, and provides context for the modifications made to the codebase. And was also generate by Simply Grok ;-)

@eriktodx
Copy link
Owner

eriktodx commented Mar 29, 2025

Lets rename tabPreview to tabWithPreview and OUTPUT_METHOD_TAB_PREVIEW to OUTPUT_METHOD_TAB_WITH_PREVIEW and its value accordingly.

Otherwise looks good. Great use of Simply Grok to improve Simply Grok. =)

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.

3 participants