Skip to content

Conversation

@alerizzo
Copy link
Contributor

Remove enigma custom instructions

@alerizzo alerizzo requested a review from a team as a code owner November 24, 2025 11:48
Copilot AI review requested due to automatic review settings November 24, 2025 11:48
Copy link

@codacy-production codacy-production bot left a comment

Choose a reason for hiding this comment

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

This PR removes the enigma-specific rules and related file checks, simplifying rule assembly to only repository and common rules. No new Codacy issues or complexity deltas were reported. Changes reduce surface area (removes filesystem checks and long guidance content) which lowers maintenance burden but may remove helpful user guidance if enigma support was previously expected.

Removing enigma rules and their file checks simplifies the code and avoids reading project-specific config (.codacy/codacy.yaml). Ensure there's a deliberate decision communicated in the PR description about dropping enigma support or moving guidance elsewhere (docs or separate generator) so users relying on that guidance are not surprised.

name: 'Codacy Rules',
description: "Configuration for AI behavior when interacting with Codacy's MCP Server",
rules: [...repositoryRules, ...commonRules, ...enigmaRules].filter((rule) => !excludedScopes?.includes(rule.scope)),
rules: [...repositoryRules, ...commonRules].filter((rule) => !excludedScopes?.includes(rule.scope)),

Choose a reason for hiding this comment

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

Good simplification: previously complex logic that read .codacy/codacy.yaml and injected a large enigma guidance block has been removed. Confirm there are no downstream features or docs that still expect the enigma guidance or the enigma detection behavior.

Copilot finished reviewing on behalf of alerizzo November 24, 2025 11:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes enigma-related custom instructions and rules from the Codacy VS Code extension.

  • Removes the logic that checks for enigma configuration in .codacy/codacy.yaml
  • Eliminates the enigma-specific rule definitions that guided AI on how to create enigma rules
  • Simplifies the rules array construction by removing the enigmaRules reference

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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