Skip to content

Conversation

Copy link
Contributor

Copilot AI commented May 22, 2025

  • Create ParameterRemover utility in src/utils/ParameterRemover.ts
    • Implement the visitor pattern for AST traversal
    • Add logic to remove parameter expressions from binary expressions
    • Add special handling for logical expressions (AND/OR)
    • Add handling for recursive traversal (subqueries, CTEs)
  • Create tests in tests/utils/ParameterRemover.test.ts
    • Add SqlComponent-level unit tests
    • Add query-level integration tests
  • Create docs/test-policy.md to document test policy
  • Update README.md to document the new feature

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot finished work on behalf of mk3008 May 22, 2025 21:46
@mk3008
Copy link
Owner

mk3008 commented May 22, 2025

@copilot まずはSqlComponentからパラメータ式を除去するコードを書いて。それができたらレビューするからPR出して。

Copilot finished work on behalf of mk3008 May 22, 2025 22:05
@mk3008 mk3008 merged commit cbc4253 into main Jun 9, 2025
1 check failed
@mk3008 mk3008 deleted the copilot/fix-83925081-5e17-4ff3-896b-4f60d45ba71b branch June 9, 2025 11:41
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