Add react utils hook useInterval

[dom-baur]

Add react utils hook useInterval from issue #6

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/neolution-ch/react-utils/@neolution-ch/react-utils@10

commit: ac3da6c

[Copilot]

Pull Request Overview

This PR introduces a new React utility hook useInterval that provides a declarative way to manage JavaScript intervals with start/stop controls and auto-start functionality.

Key changes:

• Implements a custom React hook with TypeScript interfaces for managing intervals
• Adds comprehensive Cypress component tests covering various interval scenarios
• Updates the changelog to document the new feature

Reviewed Changes

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

File	Description
src/lib/hooks/useInterval.ts	Core implementation of the useInterval hook with TypeScript interfaces
cypress/component/useInterval.cy.tsx	Comprehensive test suite covering autostart, manual control, and edge cases
CHANGELOG.md	Documents the addition of the new useInterval hook

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

This useEffect could cause infinite re-renders. The cleanup function stopInterval is returned but the effect depends on stopInterval itself, and calling startInterval inside the effect when isRunning is true could trigger the effect again. Consider restructuring to avoid this dependency cycle.

Suggested change

	console.log("Starting interval", isRunning);
	if (isRunning) {
	startInterval();
	}
	return stopInterval;
	}, [callback, isRunning, interval, startInterval, stopInterval]);
	if (isRunning) {
	if (!intervalRef.current || intervalRef.current === -1) {
	intervalRef.current = window.setInterval(() => {
	callbackRef.current();
	}, interval);
	}
	} else {
	if (intervalRef.current && intervalRef.current !== -1) {
	window.clearInterval(intervalRef.current);
	intervalRef.current = -1;
	}
	}
	return () => {
	if (intervalRef.current && intervalRef.current !== -1) {
	window.clearInterval(intervalRef.current);
	intervalRef.current = -1;
	}
	};
	}, [callback, isRunning, interval]);

[neoscie]

@neotrow is this accurate?

[neotrow]

@dom-baur

overall looking good, the word "poll" is still occuring a lot, can you replace the comments accordingly everywhere?

Thanks 🙏