Skip to content

Remove duplicate changeset prevention to allow multiple changesets#14

Merged
IzumiSy merged 6 commits intomainfrom
copilot/remove-banner-duplicate-prevention
Jan 6, 2026
Merged

Remove duplicate changeset prevention to allow multiple changesets#14
IzumiSy merged 6 commits intomainfrom
copilot/remove-banner-duplicate-prevention

Conversation

Copy link
Contributor

Copilot AI commented Jan 6, 2026

Plan to Remove Duplicate Prevention Feature

  • Remove duplicate detection and deletion logic from createChangesets function
  • Remove the recreated field from ChangesetResult interface
  • Update tests to reflect the new behavior:
    • Removed test for "should remove existing auto-generated changesets before creating new ones"
    • Removed test for "should not remove changesets without auto-generated banner"
    • Updated remaining tests to not check for recreated field
  • Remove unused imports (readChangesets, fs, path)
  • Remove unused function (isAutoGeneratedChangeset)
  • Remove unused @changesets/read dependency from package.json
  • Remove AUTO_GENERATED_BANNER constant and stop appending it to changeset summaries
  • Bump patch version from 0.2.1 to 0.2.2
  • Run all tests to ensure changes work correctly (11 tests pass)
  • Build the project to ensure no compilation errors
  • Type-check passes successfully
Original prompt

現在このCLIツールが持っているmarkdownのバナーで重複作成を防止する機能を除去して欲しいです。これだと複数のchangesetをdeps2changesets経由で作成できない(常に上書き)するようになってしまっているので。


💡 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 AI and others added 2 commits January 6, 2026 03:59
…ration

Co-authored-by: IzumiSy <982850+IzumiSy@users.noreply.github.com>
Co-authored-by: IzumiSy <982850+IzumiSy@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove duplicate prevention for markdown banner in CLI tool Remove duplicate changeset prevention to allow multiple changesets Jan 6, 2026
Copilot AI requested a review from IzumiSy January 6, 2026 04:04
@IzumiSy
Copy link
Owner

IzumiSy commented Jan 6, 2026

@copilot @changesets/read looks unused anymore. Can you remove it as well in this PR?

Co-authored-by: IzumiSy <982850+IzumiSy@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 6, 2026

@copilot @changesets/read looks unused anymore. Can you remove it as well in this PR?

Done! Removed @changesets/read from dependencies in commit 9cb4ee7.

@IzumiSy
Copy link
Owner

IzumiSy commented Jan 6, 2026

@copilot Bump patch version please

Co-authored-by: IzumiSy <982850+IzumiSy@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 6, 2026

@copilot Bump patch version please

Bumped version from 0.2.1 to 0.2.2 in commit a8a0fe3.

@IzumiSy
Copy link
Owner

IzumiSy commented Jan 6, 2026

@copilot AUTO_GENERATED_BANNER should not be used anymore. Remove it and the related code.

Co-authored-by: IzumiSy <982850+IzumiSy@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 6, 2026

@copilot AUTO_GENERATED_BANNER should not be used anymore. Remove it and the related code.

Removed AUTO_GENERATED_BANNER constant and stopped appending it to changeset summaries in commit aec8e96.

@IzumiSy IzumiSy marked this pull request as ready for review January 6, 2026 05:36
@IzumiSy IzumiSy merged commit 57c6c6e into main Jan 6, 2026
1 check passed
@IzumiSy IzumiSy deleted the copilot/remove-banner-duplicate-prevention branch January 6, 2026 05:36
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