Skip to content

fix(docs): exclude the draft hook-philosophy reference pages from the docs site - #418

Merged
mnxmnz merged 1 commit into
mainfrom
docs/exclude-draft-contributor-pages
Aug 16, 2026
Merged

fix(docs): exclude the draft hook-philosophy reference pages from the docs site#418
mnxmnz merged 1 commit into
mainfrom
docs/exclude-draft-contributor-pages

Conversation

@mnxmnz

@mnxmnz mnxmnz commented Aug 16, 2026

Copy link
Copy Markdown
Member

Overview

Closes #417

docs/hook-design-principles.md ("Status: Draft (pending discussion)") and docs/react-hook-usage-patterns.md (+ ko mirrors) are unlinked contributor drafts inside srcDir and missing from srcExclude, so every build publishes them — both are live on the production site today, and the draft is in the site's local-search index. Same class as #410 / #411, which excluded the other contributor-facing markdown.

Changes

     'CLAUDE.md',
     'AGENTS.md',
+    'docs/hook-design-principles.md',
+    'docs/ko/hook-design-principles.md',
+    'docs/react-hook-usage-patterns.md',
+    'docs/ko/react-hook-usage-patterns.md',
     'examples/**',

The documents stay in the repo; this controls exposure without prejudging their pending discussion.

Verification

  • Before: yarn docs:build (vitepress 1.6.3) emits 137 pages including all four at /docs/*.html; the local-search chunk contains the draft's anchors. Both pages serve 200 in production today (2026-08-16).
  • After: build exits 0; dist diff removes exactly the four pages (137 → 133); rebuilt search-index chunks contain zero occurrences of either slug.
  • No dead links: the four pages link only to each other; zero external referrers in the built site.
  • yarn fix is clean.

Notes

Checklist

  • Did you write the test code? — N/A, docs build config only (verified with before/after site builds)
  • Have you run yarn run fix to format and lint the code and docs?
  • Have you run yarn run test:coverage to make sure there is no uncovered line? — N/A, no source changes
  • Did you write the JSDoc? — N/A, no source changes

@changeset-bot

changeset-bot Bot commented Aug 16, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 885181c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (dfd8ed3) to head (885181c).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #418   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           58        58           
  Lines         1643      1643           
  Branches       499       499           
=========================================
  Hits          1643      1643           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 98.3 kB

ℹ️ View Unchanged
Filename Size
packages/core/dist/components/ImpressionArea/ImpressionArea.cjs 1.01 kB
packages/core/dist/components/ImpressionArea/ImpressionArea.mjs 984 B
packages/core/dist/components/Separated/Separated.cjs 686 B
packages/core/dist/components/Separated/Separated.mjs 684 B
packages/core/dist/components/SwitchCase/SwitchCase.cjs 701 B
packages/core/dist/components/SwitchCase/SwitchCase.mjs 699 B
packages/core/dist/hooks/useAsyncEffect/useAsyncEffect.cjs 615 B
packages/core/dist/hooks/useAsyncEffect/useAsyncEffect.mjs 604 B
packages/core/dist/hooks/useBooleanState/useBooleanState.cjs 539 B
packages/core/dist/hooks/useBooleanState/useBooleanState.mjs 529 B
packages/core/dist/hooks/useCallbackOncePerRender/useCallbackOncePerRender.cjs 781 B
packages/core/dist/hooks/useCallbackOncePerRender/useCallbackOncePerRender.mjs 752 B
packages/core/dist/hooks/useConditionalEffect/useConditionalEffect.cjs 945 B
packages/core/dist/hooks/useConditionalEffect/useConditionalEffect.mjs 925 B
packages/core/dist/hooks/useControlledState/useControlledState.cjs 865 B
packages/core/dist/hooks/useControlledState/useControlledState.mjs 855 B
packages/core/dist/hooks/useCounter/useCounter.cjs 1.03 kB
packages/core/dist/hooks/useCounter/useCounter.mjs 1.01 kB
packages/core/dist/hooks/useDebounce/debounce.cjs 458 B
packages/core/dist/hooks/useDebounce/debounce.mjs 456 B
packages/core/dist/hooks/useDebounce/useDebounce.cjs 974 B
packages/core/dist/hooks/useDebounce/useDebounce.mjs 953 B
packages/core/dist/hooks/useDebouncedCallback/useDebouncedCallback.cjs 1.05 kB
packages/core/dist/hooks/useDebouncedCallback/useDebouncedCallback.mjs 1.03 kB
packages/core/dist/hooks/useDoubleClick/useDoubleClick.cjs 1.08 kB
packages/core/dist/hooks/useDoubleClick/useDoubleClick.mjs 1.07 kB
packages/core/dist/hooks/useGeolocation/useGeolocation.cjs 2.14 kB
packages/core/dist/hooks/useGeolocation/useGeolocation.mjs 2.13 kB
packages/core/dist/hooks/useImpressionRef/useImpressionRef.cjs 1.2 kB
packages/core/dist/hooks/useImpressionRef/useImpressionRef.mjs 1.16 kB
packages/core/dist/hooks/useInputState/useInputState.cjs 589 B
packages/core/dist/hooks/useInputState/useInputState.mjs 579 B
packages/core/dist/hooks/useIntersectionObserver/useIntersectionObserver.cjs 1.07 kB
packages/core/dist/hooks/useIntersectionObserver/useIntersectionObserver.mjs 1.04 kB
packages/core/dist/hooks/useInterval/useInterval.cjs 837 B
packages/core/dist/hooks/useInterval/useInterval.mjs 811 B
packages/core/dist/hooks/useIsClient/useIsClient.cjs 585 B
packages/core/dist/hooks/useIsClient/useIsClient.mjs 574 B
packages/core/dist/hooks/useIsomorphicLayoutEffect/useIsomorphicLayoutEffect.cjs 577 B
packages/core/dist/hooks/useIsomorphicLayoutEffect/useIsomorphicLayoutEffect.mjs 575 B
packages/core/dist/hooks/useList/useList.cjs 850 B
packages/core/dist/hooks/useList/useList.mjs 825 B
packages/core/dist/hooks/useLoading/useLoading.cjs 913 B
packages/core/dist/hooks/useLoading/useLoading.mjs 902 B
packages/core/dist/hooks/useLongPress/useLongPress.cjs 1.79 kB
packages/core/dist/hooks/useLongPress/useLongPress.mjs 1.76 kB
packages/core/dist/hooks/useMap/useMap.cjs 730 B
packages/core/dist/hooks/useMap/useMap.mjs 713 B
packages/core/dist/hooks/useOutsideClickEffect/useOutsideClickEffect.cjs 904 B
packages/core/dist/hooks/useOutsideClickEffect/useOutsideClickEffect.mjs 878 B
packages/core/dist/hooks/usePreservedCallback/usePreservedCallback.cjs 691 B
packages/core/dist/hooks/usePreservedCallback/usePreservedCallback.mjs 675 B
packages/core/dist/hooks/usePreservedReference/usePreservedReference.cjs 794 B
packages/core/dist/hooks/usePreservedReference/usePreservedReference.mjs 777 B
packages/core/dist/hooks/usePrevious/usePrevious.cjs 633 B
packages/core/dist/hooks/usePrevious/usePrevious.mjs 625 B
packages/core/dist/hooks/useRefEffect/useRefEffect.cjs 771 B
packages/core/dist/hooks/useRefEffect/useRefEffect.mjs 744 B
packages/core/dist/hooks/useSet/useSet.cjs 822 B
packages/core/dist/hooks/useSet/useSet.mjs 791 B
packages/core/dist/hooks/useStorageState/storage.cjs 543 B
packages/core/dist/hooks/useStorageState/storage.mjs 527 B
packages/core/dist/hooks/useStorageState/useStorageState.cjs 956 B
packages/core/dist/hooks/useStorageState/useStorageState.mjs 947 B
packages/core/dist/hooks/useThrottle/throttle.cjs 306 B
packages/core/dist/hooks/useThrottle/throttle.mjs 298 B
packages/core/dist/hooks/useThrottle/useThrottle.cjs 864 B
packages/core/dist/hooks/useThrottle/useThrottle.mjs 838 B
packages/core/dist/hooks/useThrottledCallback/useThrottledCallback.cjs 1.11 kB
packages/core/dist/hooks/useThrottledCallback/useThrottledCallback.mjs 1.08 kB
packages/core/dist/hooks/useTimeout/useTimeout.cjs 627 B
packages/core/dist/hooks/useTimeout/useTimeout.mjs 600 B
packages/core/dist/hooks/useToggle/useToggle.cjs 524 B
packages/core/dist/hooks/useToggle/useToggle.mjs 506 B
packages/core/dist/hooks/useVisibilityEvent/useVisibilityEvent.cjs 769 B
packages/core/dist/hooks/useVisibilityEvent/useVisibilityEvent.mjs 749 B
packages/core/dist/index.cjs 967 B
packages/core/dist/index.mjs 697 B
packages/core/dist/utils/buildContext/buildContext.cjs 794 B
packages/core/dist/utils/buildContext/buildContext.mjs 774 B
packages/core/dist/utils/mergeProps/mergeProps.cjs 669 B
packages/core/dist/utils/mergeProps/mergeProps.mjs 668 B
packages/core/dist/utils/mergeRefs/mergeRefs.cjs 614 B
packages/core/dist/utils/mergeRefs/mergeRefs.mjs 612 B
packages/mobile/dist/hooks/useAvoidKeyboard/useAvoidKeyboard.cjs 1.01 kB
packages/mobile/dist/hooks/useAvoidKeyboard/useAvoidKeyboard.mjs 986 B
packages/mobile/dist/hooks/useBodyScrollLock/useBodyScrollLock.cjs 568 B
packages/mobile/dist/hooks/useBodyScrollLock/useBodyScrollLock.mjs 544 B
packages/mobile/dist/hooks/useKeyboardHeight/useKeyboardHeight.cjs 709 B
packages/mobile/dist/hooks/useKeyboardHeight/useKeyboardHeight.mjs 692 B
packages/mobile/dist/hooks/useNetworkStatus/useNetworkStatus.cjs 1.22 kB
packages/mobile/dist/hooks/useNetworkStatus/useNetworkStatus.mjs 1.2 kB
packages/mobile/dist/hooks/usePageVisibility/usePageVisibility.cjs 914 B
packages/mobile/dist/hooks/usePageVisibility/usePageVisibility.mjs 895 B
packages/mobile/dist/hooks/useSafeAreaInset/useSafeAreaInset.cjs 965 B
packages/mobile/dist/hooks/useSafeAreaInset/useSafeAreaInset.mjs 950 B
packages/mobile/dist/hooks/useScrollDirection/useScrollDirection.cjs 979 B
packages/mobile/dist/hooks/useScrollDirection/useScrollDirection.mjs 967 B
packages/mobile/dist/hooks/useVisualViewport/useVisualViewport.cjs 1.26 kB
packages/mobile/dist/hooks/useVisualViewport/useVisualViewport.mjs 1.25 kB
packages/mobile/dist/index.cjs 530 B
packages/mobile/dist/index.mjs 377 B
packages/mobile/dist/utils/disableBodyScrollLock/disableBodyScrollLock.cjs 706 B
packages/mobile/dist/utils/disableBodyScrollLock/disableBodyScrollLock.mjs 698 B
packages/mobile/dist/utils/enableBodyScrollLock/enableBodyScrollLock.cjs 696 B
packages/mobile/dist/utils/enableBodyScrollLock/enableBodyScrollLock.mjs 687 B
packages/mobile/dist/utils/getKeyboardHeight/getKeyboardHeight.cjs 632 B
packages/mobile/dist/utils/getKeyboardHeight/getKeyboardHeight.mjs 626 B
packages/mobile/dist/utils/getSafeAreaInset/getSafeAreaInset.cjs 925 B
packages/mobile/dist/utils/getSafeAreaInset/getSafeAreaInset.mjs 916 B
packages/mobile/dist/utils/isAndroid/isAndroid.cjs 566 B
packages/mobile/dist/utils/isAndroid/isAndroid.mjs 558 B
packages/mobile/dist/utils/isIOS/isIOS.cjs 776 B
packages/mobile/dist/utils/isIOS/isIOS.mjs 769 B
packages/mobile/dist/utils/isKeyboardVisible/isKeyboardVisible.cjs 455 B
packages/mobile/dist/utils/isKeyboardVisible/isKeyboardVisible.mjs 443 B
packages/mobile/dist/utils/isServer/isServer.cjs 404 B
packages/mobile/dist/utils/isServer/isServer.mjs 403 B
packages/mobile/dist/utils/subscribeKeyboardHeight/subscribeKeyboardHeight.cjs 1.14 kB
packages/mobile/dist/utils/subscribeKeyboardHeight/subscribeKeyboardHeight.mjs 1.12 kB

compressed-size-action

@mnxmnz
mnxmnz marked this pull request as ready for review August 16, 2026 12:55
@mnxmnz
mnxmnz requested a review from hyesungoh as a code owner August 16, 2026 12:55
@mnxmnz
mnxmnz merged commit 85d19c3 into main Aug 16, 2026
15 checks passed
@mnxmnz
mnxmnz deleted the docs/exclude-draft-contributor-pages branch August 16, 2026 12:56
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.

draft contributor reference pages are published to the docs site: hook-design-principles, react-hook-usage-patterns

2 participants