Feat/compt 30 state storage hooks

[a-elkhiraooui-ciscode]

Summary

• What does this PR change?

Why

• Why is this change needed?

Checklist

• Added/updated tests (if behavior changed)
• npm run lint passes
• npm run typecheck passes
• npm test passes
• npm run build passes
• Added a changeset (npx changeset) if this affects consumers

Notes

• Anything reviewers should pay attention to?

[sonarqubecloud]

Quality Gate passed

Issues
18 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

Pull request overview

Adds the first “state & storage” hooks to the HooksKit package, along with shared storage helpers, tests, and public exports to expose the new hooks as part of the library API.

Changes:

• Introduces useDebounce, useLocalStorage, and useSessionStorage hooks under src/hooks/.
• Adds shared SSR-guarded JSON storage helpers (readStorageValue / writeStorageValue) and Vitest coverage for the new hooks.
• Updates hook barrel exports, Copilot instructions, Husky pre-commit hook, and includes a changeset for a minor release.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/hooks/useSessionStorage.ts	Adds sessionStorage-backed state hook using shared storage helpers.
src/hooks/useSessionStorage.test.ts	Adds tests for session storage behavior + SSR helper guard.
src/hooks/useLocalStorage.ts	Adds localStorage-backed state hook using shared storage helpers.
src/hooks/useLocalStorage.test.ts	Adds tests for local storage behavior + SSR helper guard.
src/hooks/useDebounce.ts	Adds value debouncing hook using a timeout.
src/hooks/useDebounce.test.ts	Adds fake-timer tests for debounce timing behavior.
src/hooks/storage.ts	Introduces shared JSON read/write helpers with SSR guard and error swallowing.
src/hooks/index.ts	Exports the new hooks from the hooks barrel.
.husky/pre-commit	Adjusts pre-commit hook command to run lint-staged.
.github/instructions/copilot-instructions.md	Updates repo/module guidance to “HooksKit” and documents SSR-safe expectations.
.changeset/COMPT-30-state-storage-hooks.md	Declares a minor release for adding the new hooks.

[Copilot]

useDebounce uses window.setTimeout/window.clearTimeout. This will throw in non-DOM runtimes (e.g., React Native/tests without JSDOM) and contradicts the repo’s stated SSR-safety goal. Prefer globalThis.setTimeout/globalThis.clearTimeout (or the unqualified setTimeout/clearTimeout) so the hook doesn’t require window.

Suggested change

	const timeoutId = window.setTimeout(() => {
	setDebouncedValue(value);
	}, delay);
	return () => {
	window.clearTimeout(timeoutId);
	const timeoutId = setTimeout(() => {
	setDebouncedValue(value);
	}, delay);
	return () => {
	clearTimeout(timeoutId);

[Copilot]

__hooks_placeholder is exported from the hooks barrel and therefore becomes part of the package’s public API via src/index.ts. Now that real hooks are present, this placeholder export should be removed to avoid shipping a permanent, undocumented API surface.

Suggested change

	// Example placeholder export — replace with real hooks later.
	export const __hooks_placeholder = true;

[Copilot]

afterEach clears localStorage before calling vi.unstubAllGlobals(). If a test stubs window to undefined, the clear is skipped and storage can leak into later tests. Unstub/restore globals first (or use a finally) and then clear storage to keep cleanup reliable regardless of test order.

Suggested change

	if (typeof window !== 'undefined') {
	window.localStorage.clear();
	}
	vi.unstubAllGlobals();
	vi.unstubAllGlobals();
	if (typeof window !== 'undefined') {
	window.localStorage.clear();
	}

[Copilot]

afterEach clears sessionStorage before calling vi.unstubAllGlobals(). If a test stubs window to undefined, the clear is skipped and storage can leak into later tests. Unstub/restore globals first (or use a finally) and then clear storage to keep cleanup reliable regardless of test order.

Suggested change

	if (typeof window !== 'undefined') {
	window.sessionStorage.clear();
	}
	vi.unstubAllGlobals();
	vi.unstubAllGlobals();
	if (typeof window !== 'undefined') {
	window.sessionStorage.clear();
	}

[Copilot]

Accessing window.localStorage can throw (e.g., in some privacy modes or sandboxed iframes). Since this happens during render, it would crash the hook before the try/catch in readStorageValue/writeStorageValue can help. Consider wrapping the storage acquisition in a try/catch (returning undefined on failure) or centralizing it in a safe helper.

Suggested change

	export function useLocalStorage<T>(key: string, initialValue: T): [T, Dispatch<SetStateAction<T>>] {
	const storage = typeof window === 'undefined' ? undefined : window.localStorage;
	function getLocalStorage(): Storage | undefined {
	if (typeof window === 'undefined') {
	return undefined;
	}
	try {
	return window.localStorage;
	} catch {
	return undefined;
	}
	}
	export function useLocalStorage<T>(key: string, initialValue: T): [T, Dispatch<SetStateAction<T>>] {
	const storage = getLocalStorage();

[Copilot]

Accessing window.sessionStorage can throw (e.g., in some privacy modes or sandboxed iframes). Since this happens during render, it would crash the hook before the try/catch in readStorageValue/writeStorageValue can help. Consider wrapping the storage acquisition in a try/catch (returning undefined on failure) or centralizing it in a safe helper.

Suggested change

	export function useSessionStorage<T>(
	key: string,
	initialValue: T,
	): [T, Dispatch<SetStateAction<T>>] {
	const storage = typeof window === 'undefined' ? undefined : window.sessionStorage;
	function getSessionStorage(): Storage | undefined {
	if (typeof window === 'undefined') {
	return undefined;
	}
	try {
	return window.sessionStorage;
	} catch {
	return undefined;
	}
	}
	export function useSessionStorage<T>(
	key: string,
	initialValue: T,
	): [T, Dispatch<SetStateAction<T>>] {
	const storage = getSessionStorage();