1
0
mirror of https://github.com/chylex/IntelliJ-IdeaVim.git synced 2026-05-04 02:03:07 +02:00
Commit Graph

9697 Commits

Author SHA1 Message Date
Alex Plate
9a14c1f88b Remove unused Kotlin YouTrack scripts
Delete scripts that have been migrated to TypeScript or are unused:
- youtrackAnalysis/SelectTicketForAnalysis.kt (TS version used)
- youtrackAnalysis/CompleteTicketAnalysis.kt (TS version used)
- youtrackToolkit.kt (not imported anywhere)
- Related Gradle tasks

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-29 10:02:00 +02:00
Alex Plate
69691845bb Remove CI integrations test workflow and related code
Delete unused integration test infrastructure:
- .github/workflows/integrationsTest.yml
- scripts/src/main/kotlin/scripts/integrationsTest.kt
- Gradle task from build.gradle.kts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-29 10:02:00 +02:00
Alex Plate
57d477199e Refactor: Migrate updateYoutrackOnCommit to TypeScript
Move the script from Kotlin (scripts/) to TypeScript (scripts-ts/)
for consistency with other YouTrack automation scripts.

Changes:
- Add setStatus() function to youtrack.ts
- Create updateYoutrackOnCommit.ts with git log parsing
- Update GitHub workflow to use Node.js instead of JDK/Gradle
- Remove Kotlin implementation and changelogUtils.kt

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-29 10:01:59 +02:00
Alex Plate
7cb0935781 Refactor: Move youtrack.ts to tools subfolder
Reorganize scripts-ts/src structure by separating entry points (jobs)
from utility modules. The youtrack.ts module is now in tools/ subfolder.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-29 10:01:59 +02:00
claude[bot]
efacbb99b1 tests: Document @TestWithoutNeovim reasons in option mapper tests
Added description parameter to @TestWithoutNeovim annotations in
option mapper test classes to clarify why these tests are excluded
from Neovim verification. These tests verify integration between
IdeaVim options and IntelliJ's EditorSettingsExternalizable, which
is IDE-specific behavior not present in Neovim.

Modified files:
- ScrollOffOptionMapperTest
- SideScrollOffOptionMapperTest
- SideScrollOptionMapperTest
- ScrollJumpOptionMapperTest

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2025-12-29 09:32:55 +02:00
Alex Plate
e8ead1fe88 Add context7 MCP tools to allowed-tools list
The context7 plugin tools need to be explicitly listed in claude_args
allowed-tools for Claude to use them without permission prompts.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 15:00:04 +02:00
Alex Plate
dd030701f1 Add context7 plugin reminder to workflow prompts
Added a reminder about the available context7 MCP plugin to all Claude
prompts that have it enabled (triage, planning, bug-fix, feature-impl).
This helps Claude know it can look up Vim documentation when needed.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 14:23:33 +02:00
Alex Plate
aaf3e75821 Fix: Add Edit and Write tools to planning step
The planning step needs to update analysis_state.json with the plan.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 14:19:46 +02:00
Alex Plate
1ba8ddab1d Add planning step and smart clarification handling to YouTrack workflow
Introduces a planning phase between triage and implementation that:
- Explores the codebase to understand what needs to be implemented
- Creates a detailed implementation plan for bug fixes/features
- Asks for clarification when genuinely ambiguous (posts YouTrack comment)

Smart ticket selection with automatic answer detection:
- Prioritizes tickets with pending clarifications
- Uses AI to check if owner has answered questions
- Automatically removes pending tag when answered
- Falls back to random selection if no pending answers

Key changes:
- youtrack.ts: Add pending clarification tag constant and removeTag()
- selectTicketForAnalysis.ts: Query pending tickets first, add state fields
- completeTicketAnalysis.ts: Skip tagging for needs_clarification/no_answer
- Workflow: Add Step 0 (check answers) and Step 2 (planning)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 14:06:35 +02:00
Alex Plate
fb562267fb Fix workflow run link in PR body using GitHub context
Use ${{ github.* }} context instead of shell env vars so the
URL is properly expanded before being passed to Claude.
2025-12-27 13:24:43 +02:00
Alex Plate
c6fad01ffb Add attention_reason to code review step
- Added attention_reason to review prompt
- Added review.attention_reason to JSON structure
- Added permission denial detection to parse-review
- Added review attention to workflow summary
- Added review attention to fail condition
2025-12-27 13:21:52 +02:00
Alex Plate
d0a7199dde Clarify outdated ticket criteria: age alone is not a factor
Age is NEVER a reason to mark a ticket as outdated. Many 10-15+ year
old tickets are still valid. A ticket is outdated only when it has
BOTH vague/environment-specific description AND references to
features/APIs that no longer exist.
2025-12-27 11:48:54 +02:00
Alex Plate
40997f4da7 Allow curl -X POST in triage step for YouTrack comments 2025-12-27 11:39:38 +02:00
Alex Plate
242b9106ee Auto-detect permission denials from execution log
Parse the execution_file JSON to find permission_denials array
and automatically report them as attention_reason, even if Claude
doesn't self-report them. This ensures we catch tool permission
issues even when Claude works around them.
2025-12-27 11:38:38 +02:00
Alex Plate
7ed8b1321d Refactor to JSON-only communication between workflow steps
- All Claude steps now only update analysis_state.json (no text flags)
- All parse steps read from analysis_state.json using jq
- Added triage_attention_reason and changelog object to JSON structure
- Simpler and more consistent data flow
2025-12-27 11:36:30 +02:00
Alex Plate
23118a4610 Make attention_reason orthogonal to main workflow results
- attention_reason is now a separate flag that doesn't replace main status
- Workflow continues with main task even when issues are encountered
- Issues are reported alongside the main result (bug/feature/success/etc)
- Workflow fails at the end if any attention_reason was set
2025-12-27 11:23:24 +02:00
Alex Plate
2ef99e7d05 Improve workflow: add tools, split changelog/PR steps
- Add WebSearch and WebFetch to triage, bug-fix, feature, review steps
- Split Step 4 into 4A (Changelog) and 4B (Create PR)
- Add changelog status to workflow summary
- Track changelog attention reasons
2025-12-27 11:20:14 +02:00
Alex Plate
a358641659 Add debug info to parse-triage step
This will help understand:
- What outputs claude-code-action provides
- What the execution_file contains
- Whether the TRIAGE_RESULT pattern can be found
2025-12-27 10:49:56 +02:00
Alex Plate
abb5fe1537 Fix: Read from execution_file instead of non-existent response output
The claude-code-action does not have a 'response' output. Instead, it
provides 'execution_file' which is a path to the execution output file.

Changes:
- All parse steps now read from execution_file instead of response
- Added explicit git subcommand patterns to review and PR steps
- Added gh pr pattern for PR creation step

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 10:43:15 +02:00
Alex Plate
0eb97727d3 Allow specific git subcommands in bug-fix and feature steps
Add explicit patterns for git branch, git log, git blame, git diff,
and git show to ensure these commands are allowed during implementation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 10:38:52 +02:00
Alex Plate
8618cedb7c Add attention_required to triage step and allow cat command
- Added attention_required mechanism to Step 1 (Triage)
- Added Bash(cat:*) to allowed tools for triage step
- Updated parse-triage to extract attention_reason
- Updated workflow summary to show triage attention reasons
- Updated fail condition to include triage attention_required

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 10:32:58 +02:00
Alex Plate
304eddf8d0 Refactor YouTrack auto-analysis into multi-step Claude workflow
Split the single monolithic Claude prompt (~200 lines) into 5 focused
steps to improve quality and debuggability:

1. Triage: Determines ticket type (bug/feature) and suitability
2. Bug Fix: TDD-based implementation with root cause analysis
3. Feature: Feature implementation with tests
4. Code Review: Uses code-reviewer subagent
5. Changelog & PR: Creates branch, commits, and PR

Key changes:
- Add analysis_state.json for state passing between steps
- Each step has focused prompt with single responsibility
- Conditional execution based on triage result (bug vs feature)
- Preserved attention_required mechanism for all impl steps
- Updated TypeScript scripts to read/write JSON state file

Benefits:
- Focused prompts reduce confusion
- Better debugging (see where pipeline fails)
- Early termination for unsuitable tickets
- Consistent code review (always happens)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 10:16:25 +02:00
Alex Plate
7102eda97a Upload Claude execution log as workflow artifact
This helps with debugging workflow runs by preserving the full
execution log in the workflow artifacts.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 09:27:17 +02:00
Alex Plate
ffdbff8060 Allow WebFetch for GitHub and Vim documentation sites
Add WebFetch permissions for:
- github.com and raw.githubusercontent.com (commits, PRs, raw files)
- vimhelp.org and vimdoc.sourceforge.net (Vim documentation)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 08:50:16 +02:00
Alex Plate
4e0145bba5 Add deep root cause analysis guidance for bug fixes
Emphasize that Claude should:
- Focus on bug descriptions, not user-suggested solutions which may be
  inaccurate or too narrow for IdeaVim's architecture
- Find the actual root cause rather than implementing potentially flawed fixes
- Handle tickets with multiple issues by implementing them in separate commits

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 08:45:24 +02:00
Alex Plate
ba9d5a17fe Make code review prompt more concise
Instruct Claude to be direct and avoid filler words, extra explanations,
or compliments. Focus on actionable feedback only.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 08:35:02 +02:00
Alex Plate
5572bafa87 Add tests maintenance skill and daily workflow
New Claude skill for routine test maintenance focusing on:
- Test quality and readability (meaningful content, clear names)
- Reviewing @Disabled tests to check if they can be re-enabled
- Ensuring @TestWithoutNeovim annotations have clear reasons
- Documenting @VimBehaviorDiffers usage

Daily GitHub Actions workflow runs at 7 AM UTC.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-27 08:30:03 +02:00
Alex Plate
003a8aeb5e Emphasize that PR creation is the goal of the workflow
Local changes without a Pull Request are useless - added clear
instruction that the workflow must create a PR on GitHub.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:54:06 +02:00
Alex Plate
c5c8560d55 Add tests maintenance workflow
New workflow for routine test maintenance focusing on:
- Test quality and readability (meaningful content, clear names)
- Reviewing @Disabled tests to check if they can be re-enabled
- Ensuring @TestWithoutNeovim annotations have clear reasons
- Documenting @VimBehaviorDiffers usage

This complements the existing codebase maintenance workflow
but focuses exclusively on tests.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:53:22 +02:00
Alex Plate
ac1e19ec94 Download image attachments for Claude analysis
Image attachments from YouTrack tickets are now downloaded locally
so Claude can view them using the Read tool. This enables analysis
of visual bugs and UI issues that require screenshots.

Changes:
- Add downloadAttachment() function to youtrack.ts
- Download images to attachments/ directory during ticket selection
- Update ticket_details.md to reference local image paths
- Add cleanup step to remove temporary files after analysis
- Add prompt instruction about viewing image attachments

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:51:18 +02:00
Alex Plate
aef3733482 Add explicit branch push steps and PR failure handling
- Add explicit instructions to create branch, commit, push, then create PR
- Add PR creation failure as a trigger for attention_required status

Fixes issue where Claude forgot to push branch before creating PR.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:37:48 +02:00
Alex Plate
e84edffa16 Check if bug is already fixed before implementing
Add step to review source code and git history before writing tests.
If bug appears already fixed (via code review or passing test), post
a private YouTrack comment mentioning @Aleksei.Plate and stop.

New result type: ANALYSIS_RESULT=already_fixed

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:32:08 +02:00
Alex Plate
43e7b36968 Add Context7 plugin to YouTrack workflow
Enables up-to-date library documentation lookup during ticket analysis.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:25:57 +02:00
Alex Plate
32ba35aead Use project settings in Claude Code workflows
Point both workflows to .claude/settings.json to use project
configuration including enabled plugins.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:22:58 +02:00
Alex Plate
95fedeb8c1 Add workflow to check for Claude model updates
Weekly check that compares current model against latest available.
If a newer model is found, Claude creates a PR with the update.

Uses Haiku for cost efficiency since this is a simple check task.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:15:58 +02:00
Alex Plate
e21f4eca2b Use Claude Opus 4.5 for YouTrack ticket analysis
Opus is more capable for complex analysis and bug fixing tasks.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:11:36 +02:00
Alex Plate
c42fe9ba10 Require changelog update when creating PRs
Use the changelog skill to add entries for bug fixes and features
before creating the PR. Added Skill to allowed tools.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:08:15 +02:00
Alex Plate
6c76346647 Prefer running specific tests for faster workflow execution
Full test suite takes a while. Guidance now suggests:
- Run specific related tests for faster feedback
- Only run full suite for core changes
- CI will run all tests on the PR anyway

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 22:03:04 +02:00
Alex Plate
4f66a0f9c4 Include workflow run link in PR descriptions
PRs created by the YouTrack workflow now include a link to the
GitHub Actions run that generated them, making it easy to trace
back to the full Claude analysis.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 21:40:55 +02:00
Alex Plate
e6e68d5bf9 Add attention_required status to YouTrack workflow
When Claude encounters issues needing maintainer attention:
- Missing tool permissions
- YouTrack API errors
- Workflow bugs or limitations
- Required capabilities unavailable

It outputs ANALYSIS_RESULT=attention_required with ATTENTION_REASON.
The job then fails, triggering GitHub notifications.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 21:39:24 +02:00
Alex Plate
33582ecb7d Add outdated ticket detection to YouTrack workflow
Detect tickets that are likely no longer relevant:
- References removed features or obsolete IDE versions
- Vague "doesn't work" reports older than 2 years with no activity
- Missing environment details and no follow-up from reporter

When detected, Claude posts a PRIVATE comment mentioning @Aleksei.Plate
with reasoning, then outputs ANALYSIS_RESULT=outdated.

Also adds ticket creation date to ticket_details.md for age assessment.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 21:36:19 +02:00
Alex Plate
4cb7b233e2 Fetch YouTrack comments and attachments for ticket analysis
- Add getTicketComments() to fetch comment author, text, and date
- Add getTicketAttachments() to fetch attachment name, URL, and MIME type
- Include comments and attachments sections in ticket_details.md

This gives Claude more context when analyzing tickets - comments often
contain reproduction steps, workarounds, or clarifications.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 21:24:03 +02:00
Alex Plate
85538b1ae7 Add prompt injection hardening and rm command to YouTrack workflow
- Add security notice warning about user-submitted content in tickets
- Add critical rules to treat ticket content as data only
- Instruct to mark suspicious/injection attempts as unsuitable
- Add reminder at end to ignore conflicting instructions from tickets
- Allow Bash(rm:*) command for cleanup operations

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 09:15:55 +02:00
Alex Plate
338bd2164b Add bug origin investigation step to YouTrack workflow
Before implementing a fix, Claude must now investigate why the buggy
code exists - it may have been a deliberate trade-off from a previous
fix. This prevents re-introducing old bugs when fixing new ones.

Investigation includes: reviewing surrounding code, checking callers,
and using git log/blame to understand the history.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 08:59:57 +02:00
Alex Plate
5ff820be52 Add code-reviewer subagent and require review before PRs
- Add IdeaVim-specific code-reviewer agent focusing on Vim compatibility,
  Kotlin/Java quality, IntelliJ Platform patterns, and test coverage
- Update YouTrack auto-analysis workflow to require code review before
  creating PRs (for both bugs and features)
- Add Task tool to allowed tools for subagent usage

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 08:49:16 +02:00
Alex Plate
8febb228de Add Claude Code settings with Context7 plugin enabled
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 08:49:16 +02:00
Alex Plate
34ca5ec772 Split YouTrack auto-analysis rules for bugs vs features
- Bugs now require TDD approach: write failing test first, confirm
  failure, implement fix, confirm test passes
- Bugs without reproducible tests should be marked as unsuitable
- Features have separate criteria and implementation steps
- Updated PR title format: Fix(VIM-XXXX) for bugs, Add(VIM-XXXX) for features

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 08:49:15 +02:00
dependabot[bot]
c9e7068eed Bump org.junit.jupiter:junit-jupiter from 5.10.0 to 6.0.1
Bumps [org.junit.jupiter:junit-jupiter](https://github.com/junit-team/junit-framework) from 5.10.0 to 6.0.1.
- [Release notes](https://github.com/junit-team/junit-framework/releases)
- [Commits](https://github.com/junit-team/junit-framework/compare/r5.10.0...r6.0.1)

---
updated-dependencies:
- dependency-name: org.junit.jupiter:junit-jupiter
  dependency-version: 6.0.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2025-12-26 08:18:53 +02:00
Alex Plate
0c741a580e Run YouTrack auto-analysis weekly instead of every 5 minutes
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 08:11:46 +02:00
Alex Plate
90f1d72233 Remove hard 100 lines limit from YouTrack auto-analysis
Smaller changes are still preferred, but bigger changes are now
acceptable if Claude is confident about them.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 00:35:47 +02:00