Skip to content

fix: Correct typo in REPO_SYNC_CHAT_MODE_PATH constant to enable proper chatmodes synchronization#17

Merged
MatFillion merged 3 commits intomainfrom
fix/fix-chatmodes-sync
Oct 22, 2025
Merged

fix: Correct typo in REPO_SYNC_CHAT_MODE_PATH constant to enable proper chatmodes synchronization#17
MatFillion merged 3 commits intomainfrom
fix/fix-chatmodes-sync

Conversation

@MatFillion
Copy link
Member

@MatFillion MatFillion commented Oct 22, 2025

Typo was causing chatmodes not to sync.

Summary by CodeRabbit

  • Bug Fixes

    • Fixed path handling so chatmodes now sync correctly.
  • Documentation

    • Updated docs to reflect new prompt path mappings and automatic adaptation for non-default profiles.
    • Renamed authentication commands and updated troubleshooting steps.
    • Removed manual sync and view-sync-status instructions.
  • Chores

    • Package version bumped to 1.5.0.

Copilot AI review requested due to automatic review settings October 22, 2025 03:05
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 22, 2025

Walkthrough

Changed a repository constant path, updated documentation and changelog entries, and bumped package version. Specifically, REPO_SYNC_CHAT_MODE_PATH was changed from 'chatmode/' to 'chatmodes/'; README and CHANGELOG were edited; package.json version moved from 1.4.0 to 1.5.0.

Changes

Cohort / File(s) Change Summary
Constant path update
src/constant.ts
Updated exported constant REPO_SYNC_CHAT_MODE_PATH from 'chatmode/' to 'chatmodes/'.
Documentation updates
README.md, CHANGELOG.md
README reorganized prompts/path mappings, renamed Azure DevOps auth commands, removed manual sync/view status sections; CHANGELOG added "Chatmodes support" fixed item.
Version bump
package.json
Bumped version from 1.4.0 to 1.5.0.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

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 PR title "fix: Correct typo in REPO_SYNC_CHAT_MODE_PATH constant to enable proper chatmodes synchronization" accurately reflects the main change in the changeset. The core modification updates the constant REPO_SYNC_CHAT_MODE_PATH from 'chatmode/' to 'chatmodes/', which directly addresses the typo preventing chatmodes synchronization. The title is specific, clearly identifies the issue being fixed, and uses appropriate conventional prefixes. While the PR includes supporting changes like documentation updates and version bumping, the title appropriately focuses on the primary technical change, which is the expected scope for a single-sentence PR title.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/fix-chatmodes-sync

📜 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 5a62948 and 242be0e.

📒 Files selected for processing (3)
  • CHANGELOG.md (1 hunks)
  • README.md (4 hunks)
  • package.json (1 hunks)
✅ Files skipped from review due to trivial changes (1)
  • package.json
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-10-07T18:33:20.898Z
Learnt from: MatFillion
PR: nventive/Promptitude#11
File: src/utils/azureDevOps.ts:137-176
Timestamp: 2025-10-07T18:33:20.898Z
Learning: For the Promptitude extension's Azure DevOps integration, the legacy Azure DevOps URL format `https://{org}.visualstudio.com/_git/{repo}` (without a project segment) is intentionally not supported as it's considered too old. Only modern formats and legacy formats with project segments are supported.

Applied to files:

  • README.md
🔇 Additional comments (4)
CHANGELOG.md (1)

10-10: ✅ CHANGELOG entry appropriately captures the fix.

The entry clearly describes the chatmodes synchronization fix and is properly placed in the "Fixed" section. The change aligns well with the PR objective.

README.md (3)

136-138: ✅ Path mappings correctly reflect the chatmodes fix.

The updated mappings now properly reference chatmodes/ (plural), which aligns with the corrected constant. These documentation updates are essential to ensure users understand the correct directory structure for synchronized content.


81-81: Path adaptation documentation is a helpful addition.

The note about automatic path adaptation for non-default VS Code profiles is clear and supports the extension's user experience. This complements the existing configuration documentation.


158-160: Verify scope and actual code changes for this PR.

The Azure DevOps PAT command documentation updates (lines 158–160, 189) appear to be separate improvements beyond the chatmodes typo fix described in the PR objective. While these updates look correct and well-documented, they extend beyond "correcting the typo in REPO_SYNC_CHAT_MODE_PATH."

Additionally, the AI summary mentions version bump from 1.4.0 to 1.5.0 in package.json, but that file wasn't provided for review.

Please confirm:

  1. Are the Azure DevOps command changes intentional additions to this PR, or scope creep?
  2. Can you provide the actual code change in src/constant.ts showing the typo fix and the updated package.json version?

Also applies to: 189-189


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.

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 fixes a typo in the REPO_SYNC_CHAT_MODE_PATH constant, changing it from chatmode/ to chatmodes/ (plural). This correction ensures that the extension properly synchronizes chat mode files from the expected repository directory structure.

Key Changes

  • Corrected the path constant from chatmode/ to chatmodes/ to match the actual directory naming convention used in repositories

Copy link
Contributor

@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 740880f and 5a62948.

📒 Files selected for processing (1)
  • src/constant.ts (1 hunks)

@MatFillion MatFillion merged commit 8f8886e into main Oct 22, 2025
4 checks passed
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