Improve setInterval/setTimeout checks (E5004/E5005) to scan all files and suggest proper usage

[Copilot]

Related to #182

Overview

This PR significantly improves the setInterval/setTimeout checks (E5004/E5005) to provide better coverage and more helpful guidance to adapter developers. The checks now scan all executable files, intelligently ignore proper usage patterns, and provide context-aware suggestions.

Changes

Previous Behavior

• Only checked the main file (package.json.main) if it ended with .js
• Used simple string search that couldn't distinguish between plain calls and prefixed calls
• Did not support TypeScript files
• No helpful suggestions for developers

New Behavior

• Comprehensive file scanning: Checks all .js, .ts, .mjs, and .cjs files in the repository
• Smart detection: Ignores this.setInterval(), adapter.setInterval(), this.setTimeout(), and adapter.setTimeout() calls (which are the correct usage patterns)
• Context-aware suggestions: Analyzes code structure to suggest this.* when inside adapter class or adapter.* when outside
• Better error messages: Provides specific recommendations on how to fix the issue

Error/Warning Codes

Code	Severity	Condition
E5004	Error	Plain setInterval() without clearInterval() in compact mode
W5004	Warning	Plain setInterval() without clearInterval() in non-compact mode
S5004	Suggestion	Plain setInterval() with clearInterval() present
W5005	Warning	Plain setTimeout() without clearTimeout()
S5005	Suggestion	Plain setTimeout() with clearTimeout() present

Example Output

Before:

[E5004] setInterval found in "main.js", but no clearInterval detected

After:

[E5004] setInterval found in "/main.js", but no clearInterval detected. Use this.setInterval() instead.
[S5004] Consider using adapter.setInterval() instead of plain setInterval() in "/lib/helper.js".
[W5005] setTimeout found in "/lib/utils.ts", but no clearTimeout detected. Use adapter.setTimeout() instead.

Implementation Details

• Reuses the existing isInAdapterClassContext() function to determine proper context
• Maintains backward compatibility with existing error code numbers
• Follows the same pattern as the deprecated method checks
• Excludes files in: node_modules, admin, admin-src, .git, .vscode, .dev-server

Testing

• ✅ All ESLint checks pass
• ✅ Regex patterns tested for correct detection of plain vs prefixed calls
• ✅ Logic verified for context-aware suggestions
• ✅ No breaking changes to existing functionality

Impact

This improvement helps adapter developers:

• Prevent memory leaks by ensuring proper timer cleanup
• Follow best practices with adapter timer methods
• Get immediate, actionable feedback with context-aware suggestions
• Support TypeScript development with proper .ts file checking

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• data.jsdelivr.com
  • Triggering command: node (dns block)
• https://api.github.com/repos/ioBroker/ioBroker.javascript
  • Triggering command: node index.js REDACTED --local --debug (http block)
• https://api.github.com/repos/ioBroker/ioBroker.test
  • Triggering command: node /home/REDACTED/work/ioBroker.repochecker/ioBroker.repochecker/index.js REDACTED --local (http block)
• https://api.github.com/repos/undefined/git/trees/undefined
  • Triggering command: node (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Improve setInterval / setTimeout checks ([E504/E505]</issue_title>
<issue_description>- [ ] setInterval / setTimeout checks should be executed for .ts files too

• check should scan all .ts and .js files
• check should ignore this.setXxx entries
• check should suggest this.Xxx usage (and guess if this. or adapter. is better suited)
• if this./adapter.XXX is handled setTimeout should be error if compact is set (is temp disabled)</issue_description>

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

Fixes #236

---

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