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

9718 Commits

Author SHA1 Message Date
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
Alex Plate
efd7eee31b Run YouTrack auto-analysis 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 00:24:43 +02:00
Alex Plate
6e3c8cd1ed Fix YouTrack query syntax for Area exclusion
Use braces syntax: Area: -{Remote Dev} instead of Area: -"Remote Dev"

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 00:16:39 +02:00
Alex Plate
e844a713f9 Add private comment support and filter out Remote Dev/Gateway tickets
- Comments now use LimitedVisibility with JetBrains team group (10-3)
- Query excludes tickets with Area "Remote Dev" or "Gateway"

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 00:14:23 +02:00
Alex Plate
1d9e01e4ae [Experiment] Migrate YouTrack analysis scripts to TypeScript
Replaces Kotlin/Gradle scripts with TypeScript for the auto-analysis workflow.

New files in scripts-ts/:
- youtrack.ts: API utilities (getTicketsByQuery, getTicketDetails, setTag, addComment)
- selectTicketForAnalysis.ts: Selects random open ticket
- completeTicketAnalysis.ts: Tags ticket, optionally adds comment

Benefits:
- Faster startup (no Gradle daemon)
- Simpler dependencies (just tsx)
- Easier to read and modify

The Kotlin scripts remain for other workflows.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-26 00:04:28 +02:00
Alex Plate
4d286f7b12 Fix: Only comment on YouTrack for explicit suitable/error results
Changed condition from `!= "unsuitable"` to `== "suitable" || == "error"`
so that unknown/empty parsing results don't trigger comments.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-25 23:57:38 +02:00
Alex Plate
a1e5b67889 Fix URL encoding in getYoutrackTicketsByQuery
Use ktor's parameter() function to properly URL-encode query parameters
instead of string concatenation. This fixes 400 Bad Request errors when
queries contain spaces or special characters.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-25 23:51:43 +02:00
Alex Plate
402b81c3c9 Add experimental YouTrack auto-analysis workflow with Claude
This adds a GitHub Action that automatically analyzes random open YouTrack
tickets to determine if they're suitable for automated fixing by Claude Code.

New files:
- youtrackAutoAnalysis.yml: Manual-trigger workflow that orchestrates the process
- SelectTicketForAnalysis.kt: Picks random open ticket without "claude-analyzed" tag
- CompleteTicketAnalysis.kt: Tags ticket and optionally adds analysis comment

Changes:
- youtrack.kt: Add getTicketDetails() function and claudeAnalyzedTagId constant
- build.gradle.kts: Register selectTicketForAnalysis and completeTicketAnalysis tasks

The workflow evaluates tickets based on: clarity, reproducibility (for bugs),
scope (no big refactoring), and testability. If suitable, Claude creates a PR.
All analyzed tickets are tagged to exclude from future runs.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-25 23:45:10 +02:00
Alex Plate
ef2e069b93 Migrate PR verification from TeamCity to GitHub Actions
Replace TeamCity PR checks with a GitHub Actions workflow that:
- Runs on pull requests targeting master
- Executes the same test command with identical environment variables
- Uses Amazon Corretto JDK 21 (matching TeamCity setup)

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 09:27:45 +02:00
Alex Plate
9d829b2011 Revert Slack notification experiment, re-enable VCS triggers
- Remove SlackNotificationTest build configuration
- Remove Slack notification steps from ReleaseEap and ReleasePlugin
- Remove ANTHROPIC_API_KEY parameter from release jobs
- Keep slackUrl parameter for future use
- Re-enable VCS triggers on all test configurations

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 09:18:10 +02:00
Alex Plate
7a087fa650 Add Slack URL debug output
Check if ORG_GRADLE_PROJECT_slackUrl is set and add verbose curl output.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 09:09:54 +02:00
aleksei.plate@jetbrains.com
cf0d6b359a parameters of 'Test Slack Notification' build configuration were updated (TeamCity change in 'Ideavim' project) 2025-12-24 07:08:57 +00:00
Alex Plate
f612dcc528 Fix shell compatibility in debug step
Add #!/bin/bash shebang for bash-specific syntax.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 09:04:12 +02:00
Alex Plate
bb843511ca Add API key debug step to Slack notification test
Check if ANTHROPIC_API_KEY is set and validate it with a simple API call.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 09:01:01 +02:00
Alex Plate
95cedf441d Disable VCS triggers and add Slack notification test build
Temporarily disable VCS triggers to save TeamCity credits:
- GitHub PR checks
- TestingBuildType (all test configs)
- PropertyBased, RandomOrderTests, PluginVerifier, Nvim

Add SlackNotificationTest build to test changelog generation
and Slack notification flow separately.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 08:28:38 +02:00
Alex Plate
36680d0b7a Allow curl command in Slack notification step
Add --allowedTools "Bash(curl:*)" to Claude CLI invocation.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 22:06:28 +02:00
Alex Plate
9dd8cfa72d Update Slack notification prompts to send immediately
Don't ask for confirmation before sending the Slack notification.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 19:09:42 +02:00
Alex Plate
fe95c6dca3 Fix YouTrack token pattern and remove test configuration
- Fix ReleaseEap to use env.ORG_GRADLE_PROJECT_youtrackToken pattern
  (matching ReleasePlugin) instead of env.YOUTRACK_TOKEN
- Remove YouTrack test configuration (no longer needed)

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:59:21 +02:00
Alex Plate
347198aad3 Fix YouTrack token passing to gradle task
Use ORG_GRADLE_PROJECT_youtrackToken env var which Gradle
automatically converts to the youtrackToken project property.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:49:15 +02:00
Alex Plate
970fa15794 Add YouTrack connection test build configuration
- Add new TeamCity build type to test YouTrack API connection
- Create testYoutrackConnection gradle task
- Update YouTrack token in ReleaseEap (apply UI patch)
- Fail immediately if YOUTRACK_TOKEN is not set

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:44:58 +02:00
Alex Plate
6b52f118bc Remove vim-engine publish and Xorg test configurations
These build configurations are no longer needed.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:33:42 +02:00
aleksei.plate@jetbrains.com
6a2ac714f5 parameters of 'Publish EAP Build' build configuration were updated (TeamCity change in 'Ideavim / IdeaVim releases' project) 2025-12-23 16:31:50 +00:00
Alex Plate
e0dd804197 Use XLarge agent for EAP release build
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:30:57 +02:00
Alex Plate
dcc4d78812 Inline agent requirements to individual TeamCity build types
Move the agent CPU and OS requirements from the abstract IdeaVimBuildType
class to each build type individually. This allows per-build-type control
over agent size (MEDIUM vs XLARGE).

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:29:27 +02:00
Alex Plate
e1ae5c19c8 Switch TeamCity agents from XLarge to Medium
Add AgentSize constants for CPU count configuration and change
default agent requirement from 16 CPUs (XLarge) to 4 CPUs (Medium).

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 18:03:49 +02:00
Alex Plate
c3b7462028 Add Slack notification to EAP release build
Adds Claude Code CLI integration to send Slack notifications after EAP
releases. Uses the same approach as the main release build, with an
EAP-specific prompt that includes unreleased changelog entries.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 17:33:02 +02:00
Alex Plate
12b65fa17f Clean up Slack notification code, add ANTHROPIC_API_KEY to release
- Remove SlackNotificationTest build config and patch
- Remove unused Kotlin slackNotification code and gradle tasks
- Add ANTHROPIC_API_KEY credential to release builds
- Update prompt for calm, professional internal team tone

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 17:28:44 +02:00
Alex Plate
19cd56fa54 Move Slack notification logic to Claude Code prompt in TeamCity
Instead of using Kotlin code to orchestrate Claude Code calls,
call Claude Code directly from TeamCity with a comprehensive prompt
that handles:
- Reading CHANGES.md
- Generating Slack message JSON
- Sending to webhook (with retry on error)

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 17:18:36 +02:00
aleksei.plate@jetbrains.com
78b066abe0 parameters of 'Test Slack Notification' build configuration were updated (TeamCity change in 'Ideavim / IdeaVim releases' project) 2025-12-23 15:09:05 +00:00
Alex Plate
ff64824840 Add 60 second timeout to Claude Code CLI call
Prevents the build from hanging indefinitely if Claude Code
doesn't respond.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-23 17:07:20 +02:00