Skip to content

Conversation

@nanotaboada
Copy link
Owner

@nanotaboada nanotaboada commented Dec 10, 2025

  • Replace Docker extension: ms-azuretools.vscode-docker (official)
  • Add unwanted recommendations to prevent conflicts
  • Remove github.copilot (unified into copilot-chat)
  • Block docker.docker and vscode-containers (use official Docker)
  • Organize extensions by relevance (core dev tools first)

Summary by CodeRabbit

  • Chores
    • Reorganized the project's recommended VS Code extensions into categorized, annotated groups to better reflect current tooling and workflows.
    • Added an explicit list of unwanted extensions to deter unnecessary installs and streamline the development environment.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 10, 2025

Walkthrough

Replaces the flat VS Code recommendations list with a categorized, commented block, adds new recommended extensions including csharpier.csharpier-vscode and several infrastructure/workflow extensions, reorganizes retained items, and introduces a top-level unwantedRecommendations array containing ["github.copilot", "docker.docker"].

Changes

Cohort / File(s) Summary
VS Code Extensions Configuration
.vscode/extensions.json
Replaced the flat recommendations array with a categorized, annotated recommendations block (e.g., C# Development, Infrastructure & Workflow), added csharpier.csharpier-vscode plus multiple infra/workflow extensions, reorganized existing entries, and added a top-level unwantedRecommendations property with ["github.copilot", "docker.docker"].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Review .vscode/extensions.json for JSON validity and confirm the intended final sets for recommendations and unwantedRecommendations.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: optimize workspace extension recommendations' directly aligns with the main change of reorganizing VS Code extensions by categorizing them and adding unwanted recommendations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/optimize-vscode-extensions

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7b43a7a and f0119b3.

📒 Files selected for processing (1)
  • .vscode/extensions.json (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: coverage (codecov)
  • GitHub Check: Codacy Static Code Analysis
  • GitHub Check: Analyze (csharp)
🔇 Additional comments (1)
.vscode/extensions.json (1)

1-23: Configuration correctly aligns with modern Docker and Container Tools ecosystem.

The extension recommendations and unwanted list properly address the critical Docker configuration issue flagged in the previous review. The code now:

  • Recommends ms-azuretools.vscode-containers (Microsoft's current Container Tools extension), which provides Dev Container integration and modern container management;
  • Blocks docker.docker (Docker DX), which is now integrated into Container Tools;
  • Removes any blocking of ms-dotnettools.csharp, allowing C# Dev Kit to function with its required dependency;
  • Deprecates github.copilot (base) in favor of github.copilot-chat (the unified conversational AI interface).

The categorized inline comments are clear and helpful, improving maintainability. The organization by relevance (C# dev tools first, then infrastructure/workflow) is logical.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codacy-production
Copy link

codacy-production bot commented Dec 10, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1ffee5b) 1225 196 16.00%
Head commit (7b43a7a) 1225 (+0) 196 (+0) 16.00% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#326) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ffee5b and 7e99e3c.

📒 Files selected for processing (1)
  • .vscode/extensions.json (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Codacy Static Code Analysis
🔇 Additional comments (1)
.vscode/extensions.json (1)

16-21: unwantedRecommendations field implementation looks good.

The addition of the unwantedRecommendations array is well-structured and aligns with the PR objectives:

  • Blocks docker.docker (to enforce official Docker extension)
  • Blocks ms-azuretools.vscode-containers (deprecated in favor of official Docker)
  • Blocks github.copilot (unified into copilot-chat)
  • Blocks ms-dotnettools.csharp (see earlier concern)


- Organize extensions into logical groups with inline documentation
- Add unwanted recommendations for deprecated alternatives
- Curate 12 essential extensions (2 C# dev, 10 infrastructure/workflow)
@nanotaboada nanotaboada force-pushed the feature/optimize-vscode-extensions branch from 7e99e3c to 7b43a7a Compare December 10, 2025 17:07
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7e99e3c and 7b43a7a.

📒 Files selected for processing (1)
  • .vscode/extensions.json (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Codacy Static Code Analysis
  • GitHub Check: Analyze (csharp)

@nanotaboada
Copy link
Owner Author

@coderabbitai configuration

@coderabbitai
Copy link

coderabbitai bot commented Dec 10, 2025

CodeRabbit configuration
language: en-US
tone_instructions: ''
early_access: true
enable_free_tier: true
reviews:
  profile: chill
  request_changes_workflow: false
  high_level_summary: true
  high_level_summary_instructions: ''
  high_level_summary_placeholder: '@coderabbitai summary'
  high_level_summary_in_walkthrough: false
  auto_title_placeholder: '@coderabbitai'
  auto_title_instructions: ''
  review_status: true
  commit_status: true
  fail_commit_status: false
  collapse_walkthrough: false
  changed_files_summary: true
  sequence_diagrams: true
  estimate_code_review_effort: true
  assess_linked_issues: true
  related_issues: true
  related_prs: false
  suggested_labels: false
  labeling_instructions: []
  auto_apply_labels: false
  suggested_reviewers: false
  auto_assign_reviewers: false
  in_progress_fortune: true
  poem: false
  path_filters: []
  path_instructions: []
  abort_on_close: true
  disable_cache: false
  auto_review:
    enabled: true
    auto_incremental_review: true
    ignore_title_keywords: []
    labels: []
    drafts: false
    base_branches: []
    ignore_usernames: []
  finishing_touches:
    docstrings:
      enabled: true
    unit_tests:
      enabled: true
  pre_merge_checks:
    docstrings:
      mode: warning
      threshold: 80
    title:
      mode: warning
      requirements: ''
    description:
      mode: warning
    issue_assessment:
      mode: warning
    custom_checks: []
  tools:
    ast-grep:
      rule_dirs: []
      util_dirs: []
      essential_rules: true
      packages: []
    shellcheck:
      enabled: true
    ruff:
      enabled: true
    markdownlint:
      enabled: true
    github-checks:
      enabled: true
      timeout_ms: 90000
    languagetool:
      enabled: true
      enabled_rules: []
      disabled_rules: []
      enabled_categories: []
      disabled_categories: []
      enabled_only: false
      level: default
    biome:
      enabled: true
    hadolint:
      enabled: true
    swiftlint:
      enabled: true
    phpstan:
      enabled: true
      level: default
    phpmd:
      enabled: true
    phpcs:
      enabled: true
    golangci-lint:
      enabled: true
    yamllint:
      enabled: true
    gitleaks:
      enabled: true
    checkov:
      enabled: true
    detekt:
      enabled: true
    eslint:
      enabled: true
    flake8:
      enabled: true
    rubocop:
      enabled: true
    buf:
      enabled: true
    regal:
      enabled: true
    actionlint:
      enabled: true
    pmd:
      enabled: true
    clang:
      enabled: true
    cppcheck:
      enabled: true
    semgrep:
      enabled: true
    circleci:
      enabled: true
    clippy:
      enabled: true
    sqlfluff:
      enabled: true
    prismaLint:
      enabled: true
    pylint:
      enabled: true
    oxc:
      enabled: true
    shopifyThemeCheck:
      enabled: true
    luacheck:
      enabled: true
    brakeman:
      enabled: true
    dotenvLint:
      enabled: true
    htmlhint:
      enabled: true
    checkmake:
      enabled: true
    osvScanner:
      enabled: true
chat:
  art: true
  auto_reply: true
  integrations:
    jira:
      usage: disabled
    linear:
      usage: disabled
knowledge_base:
  opt_out: false
  web_search:
    enabled: true
  code_guidelines:
    enabled: true
    filePatterns: []
  learnings:
    scope: auto
  issues:
    scope: auto
  jira:
    usage: disabled
    project_keys: []
  linear:
    usage: disabled
    team_keys: []
  pull_requests:
    scope: auto
  mcp:
    usage: auto
    disabled_servers: []
code_generation:
  docstrings:
    language: en-US
    path_instructions: []
  unit_tests:
    path_instructions: []
issue_enrichment:
  auto_enrich:
    enabled: true
  planning:
    enabled: true
  labeling:
    labeling_instructions: []
    auto_apply_labels: false

@nanotaboada
Copy link
Owner Author

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Dec 10, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

- Replace vscode-docker with vscode-containers (modern Container Tools)
@sonarqubecloud
Copy link

@codacy-production
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1ffee5b) 1225 196 16.00%
Head commit (f0119b3) 1225 (+0) 196 (+0) 16.00% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#326) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@nanotaboada nanotaboada merged commit 44d1775 into master Dec 10, 2025
16 checks passed
@nanotaboada nanotaboada deleted the feature/optimize-vscode-extensions branch December 10, 2025 17:31
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.

2 participants