Skip to content

fix: drop HELPER_ prefix from workScopeTag constants - #122

Merged
aspiers merged 2 commits into
hypercerts-org:developfrom
aspiers:drop-helper-prefix
Jan 22, 2026
Merged

fix: drop HELPER_ prefix from workScopeTag constants#122
aspiers merged 2 commits into
hypercerts-org:developfrom
aspiers:drop-helper-prefix

Conversation

@aspiers

@aspiers aspiers commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

The helper directory was being included in generated constant names,
producing HELPER_WORK_SCOPE_TAG_* constants. This change excludes
the HELPER prefix for cleaner export names.

Summary by CodeRabbit

  • Breaking Changes

    • Removed the HELPER_ prefix from workScopeTag constants, simplifying naming conventions
  • Chores

    • Updated developer documentation to streamline the changeset creation workflow
    • Modified how export names are generated for files in helper directories

✏️ Tip: You can customize this high-level summary in your review settings.

The helper directory was being included in generated constant names,
producing HELPER_WORK_SCOPE_TAG_* constants. This change excludes
the 'helper' directory prefix for cleaner export names.
@changeset-bot

changeset-bot Bot commented Jan 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4e9ada9

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

This PR includes changesets to release 1 package
Name Type
@hypercerts-org/lexicon Minor

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

@coderabbitai

coderabbitai Bot commented Jan 22, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The pull request removes the HELPER_ prefix from three workScopeTag constants, updates developer guidance to use the writing-changesets skill instead of manual changeset workflow, and modifies the export generation script to treat "helper" subdirectories as non-meaningful alongside existing non-meaningful directories.

Changes

Cohort / File(s) Summary
Changelog entry
.changeset/drop-helper-prefix.md
Documents a minor release dropping the HELPER_ prefix from WORK_SCOPE_TAG_NSID, WORK_SCOPE_TAG_LEXICON_JSON, and WORK_SCOPE_TAG_LEXICON_DOC constants.
Developer guidance
AGENTS.md
Replaces manual changeset workflow instructions with guidance to use the writing-changesets skill. Removes detailed changeset format, version bump rules, and release flow sections; streamlines changeset generation requirements.
Export generation logic
scripts/generate-exports.js
Updates pathToImportName function to treat "helper" subdirectories as non-meaningful (alongside "claim" and "certified"), causing files in "helper" directories to use base-name-based fallback instead of subdirectory prefix.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • bitbeckers
  • s-adamantine

Poem

🐰 Helpers shed their prefix crown,
Helper dirs blend smoothly down,
Changesets flow with skill so neat—
Evolution, clean and sweet! ✨

✨ Finishing touches
  • 📝 Generate docstrings

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.

@aspiers
aspiers force-pushed the drop-helper-prefix branch from 774f3a6 to 4e9ada9 Compare January 22, 2026 02:28
@aspiers
aspiers merged commit 904a6a7 into hypercerts-org:develop Jan 22, 2026
3 checks passed
@aspiers
aspiers deleted the drop-helper-prefix branch January 22, 2026 02:29
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.

1 participant