Skip to content

Sort bookmark names alphabetically and add unit tests#36

Merged
danieldelcore merged 1 commit into
masterfrom
codex/list-issues-on-the-repository
Apr 21, 2026
Merged

Sort bookmark names alphabetically and add unit tests#36
danieldelcore merged 1 commit into
masterfrom
codex/list-issues-on-the-repository

Conversation

@danieldelcore
Copy link
Copy Markdown
Owner

Motivation

  • Ensure bookmarks are displayed and picked in a predictable, user-friendly alphabetical order.

Description

  • Add bin/bookmarks.js exporting sortBookmarksByName that performs a locale-aware, case-insensitive, numeric-aware sort of bookmark entries.
  • Use sortBookmarksByName in listBookmarks and pickAndRunBookmark inside bin/index.js so listed names and the interactive picker are sorted.
  • Add unit tests in test/bookmarks.test.mjs to validate alphabetical, case-insensitive, and numeric ordering behavior.
  • Add a changeset file .changeset/silent-stingrays-flash.md documenting the patch.

Testing

  • Added unit tests for sortBookmarksByName and ran the test suite with vitest; the new tests passed.
  • Ran npm test (which invokes the automated test runner) and the test run succeeded.

Codex Task

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 21, 2026

🦋 Changeset detected

Latest commit: 0996c36

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
scriptpal Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@danieldelcore danieldelcore merged commit 0de2bd7 into master Apr 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant