🧑‍💻 Lint typescript

[Friedinger]

This pull request updates the linting and formatting scripts in the ecoscan_app/package.json file to include a TypeScript type check. This ensures that type errors are caught during both linting and automatic fixing.

Build and linting improvements:

• Updated the lint and fix scripts to run tsc --noEmit in addition to Prettier and Expo lint, so TypeScript type errors are now checked as part of the linting process.

[coderabbitai]

Walkthrough

The Expo app’s lint and fix scripts now run TypeScript typechecking, and tsconfig.json enables additional compiler diagnostics for safer and more consistent code validation.

Changes

TypeScript validation

Layer / File(s)	Summary
TypeScript validation configuration ecoscan_app/package.json, ecoscan_app/tsconfig.json	Lint and fix scripts now run tsc --noEmit, and additional strict compiler checks are enabled.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• RubberDuckCrew/EcoScan#18: Also updates the Expo app’s TypeScript configuration.
• RubberDuckCrew/EcoScan#38: Also modifies the app’s lint and fix scripts.

Suggested labels: ✨ Enhancement

Suggested reviewers: Sophie235

Poem

I’m a rabbit checking types with care,
No hidden errors left in there.
Lint and fixes hop in line,
Strict checks make the codebase shine.
Squeak, compile, and all is fine!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is related to the change, but it is a bit generic and only mentions TypeScript linting.
Description check	✅ Passed	The description accurately matches the script and TypeScript type-check changes in the pull request.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ci/lint-typescript

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

🧹 Nitpick comments (1)

ecoscan_app/tsconfig.json (1)

10-10: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

noImplicitAny is redundant with strict: true.

strict: true (line 4) already enables noImplicitAny. This line can be removed for clarity, though it's harmless.

♻️ Remove redundant flag

     "noUnusedParameters": true,
-    "noImplicitAny": true,
     "noFallthroughCasesInSwitch": true,

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@ecoscan_app/tsconfig.json` at line 10, Remove the redundant "noImplicitAny":
true compiler option from the TypeScript configuration, since strict: true
already enables it.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@ecoscan_app/tsconfig.json`:
- Line 10: Remove the redundant "noImplicitAny": true compiler option from the
TypeScript configuration, since strict: true already enables it.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 604c179a-9483-46fd-a237-dd130fe1d354

📥 Commits

Reviewing files that changed from the base of the PR and between 916787e and 38abbb1.

📒 Files selected for processing (2)

• ecoscan_app/package.json
• ecoscan_app/tsconfig.json