Skip to content

Comments

refactor: extract shared utilities for URL building, prompts, and name resolution#18

Merged
lollipop-onl merged 1 commit intomainfrom
claude/refactor-project-docs-PXbLI
Feb 7, 2026
Merged

refactor: extract shared utilities for URL building, prompts, and name resolution#18
lollipop-onl merged 1 commit intomainfrom
claude/refactor-project-docs-PXbLI

Conversation

@lollipop-onl
Copy link
Member

  • Add resolveByName() generic factory in resolve.ts to eliminate
    duplicated name-to-ID lookup patterns (used by resolvePriorityId,
    resolveResolutionId)
  • Add url.ts with centralized Backlog web URL builders (issueUrl,
    projectUrl, pullRequestUrl, repositoryUrl, wikiUrl, dashboardUrl)
    replacing scattered template literals across 8 command files
  • Add prompt.ts with promptRequired() helper replacing duplicated
    prompt-then-validate-then-exit pattern across 15+ command files
  • Apply new utilities to all affected command files (browse, issue/,
    pr/
    , project/, wiki/, repo/, category/, team/, milestone/,
    webhook/, status-type/, issue-type/*)
  • Add comprehensive tests for all new utilities (url.test.ts,
    prompt.test.ts, resolveByName tests in resolve.test.ts)
  • Update claude.md to document new utility modules, URL builder
    patterns, and promptRequired usage

https://claude.ai/code/session_018Xzap56B6tbcxdSx2DWuq1

…e resolution

- Add resolveByName<T>() generic factory in resolve.ts to eliminate
  duplicated name-to-ID lookup patterns (used by resolvePriorityId,
  resolveResolutionId)
- Add url.ts with centralized Backlog web URL builders (issueUrl,
  projectUrl, pullRequestUrl, repositoryUrl, wikiUrl, dashboardUrl)
  replacing scattered template literals across 8 command files
- Add prompt.ts with promptRequired() helper replacing duplicated
  prompt-then-validate-then-exit pattern across 15+ command files
- Apply new utilities to all affected command files (browse, issue/*,
  pr/*, project/*, wiki/*, repo/*, category/*, team/*, milestone/*,
  webhook/*, status-type/*, issue-type/*)
- Add comprehensive tests for all new utilities (url.test.ts,
  prompt.test.ts, resolveByName tests in resolve.test.ts)
- Update claude.md to document new utility modules, URL builder
  patterns, and promptRequired usage

https://claude.ai/code/session_018Xzap56B6tbcxdSx2DWuq1
@lollipop-onl lollipop-onl merged commit 1c92313 into main Feb 7, 2026
3 checks passed
@lollipop-onl lollipop-onl deleted the claude/refactor-project-docs-PXbLI branch February 8, 2026 03:55
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