Skip to content

Add @example JSDoc tags to all hooks - #591

Merged
timowestnosto merged 3 commits into
masterfrom
copilot/add-hook-usage-examples
Nov 3, 2025
Merged

Add @example JSDoc tags to all hooks#591
timowestnosto merged 3 commits into
masterfrom
copilot/add-hook-usage-examples

Conversation

Copilot AI commented Oct 25, 2025

Copy link
Copy Markdown
Contributor

Add usage examples for all hooks using @example tags

Task Completion Summary:

All hooks now have comprehensive @example tags with realistic usage patterns
Updated JSDoc @group tags to "Hooks" for consistency

Core Infrastructure Hooks - COMPLETED:

  • useLoadClientScript - Added examples + updated @group to "Hooks"
  • useNostoApi - Added examples + updated @group to "Hooks"
  • useNostoContext - Added examples + updated @group to "Hooks"
  • useDeepCompareEffect - Added examples + updated @group to "Hooks"
  • useRenderCampaigns - Added examples + updated @group to "Hooks"

Page-specific Hooks - COMPLETED:

  • useNostoProduct - Added comprehensive examples (@group already "Hooks")
  • useNostoCategory - Added comprehensive examples (@group already "Hooks")
  • useNostoHome - Added comprehensive examples (@group already "Hooks")
  • useNostoCheckout - Added comprehensive examples (@group already "Hooks")
  • useNosto404 - Added comprehensive examples (@group already "Hooks")
  • useNostoSearch - Added comprehensive examples (@group already "Hooks")
  • useNostoOrder - Added comprehensive examples (@group already "Hooks")
  • useNostoOther - Added comprehensive examples (@group already "Hooks")
  • useNostoSession - Added comprehensive examples (@group already "Hooks")

Recent Updates (Commit 39a5642):

  • Standardized JSDoc @group tags: Changed "Essential Functions" and "Utilities" to "Hooks" for all hook functions
  • Addressed reviewer feedback: All hooks now consistently grouped under "Hooks" category

Technical Validation:

  • TypeScript compilation: ✅ No errors
  • ESLint validation: ✅ No errors
  • Documentation consistency: ✅ All hooks properly categorized as "Hooks"
Original prompt

This section details on the original issue you should resolve

<issue_title>Add usage examples for all hooks</issue_title>
<issue_description>Add usage examples for all hooks, use @example tags for that</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Add usage examples for all hooks Add @example JSDoc tags to all hooks Oct 25, 2025
Copilot AI requested a review from timowestnosto October 25, 2025 12:24

@timowestnosto timowestnosto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot fix these

Comment thread src/hooks/useLoadClientScript.ts Outdated
Comment thread src/hooks/useDeepCompareEffect.ts Outdated
Comment thread src/hooks/useNostoApi.ts Outdated
Comment thread src/hooks/useNostoContext.ts Outdated
Comment thread src/hooks/useRenderCampaigns.tsx Outdated
@timowestnosto
timowestnosto marked this pull request as ready for review October 26, 2025 10:58

Copilot AI commented Oct 26, 2025

Copy link
Copy Markdown
Contributor Author

@copilot fix these

Fixed all @group tags in commit 39a5642. Changed "Essential Functions" and "Utilities" to "Hooks" for consistent categorization across all hook functions.

@timowestnosto
timowestnosto merged commit 808f1f5 into master Nov 3, 2025
1 check passed
@timowestnosto
timowestnosto deleted the copilot/add-hook-usage-examples branch November 3, 2025 12:46
@manikandan-ravikumar
manikandan-ravikumar restored the copilot/add-hook-usage-examples branch November 17, 2025 12:16
@manikandan-ravikumar
manikandan-ravikumar deleted the copilot/add-hook-usage-examples branch November 17, 2025 12:23
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.

Add usage examples for all hooks

3 participants