fix: six real bugs the old TypeScript config was hiding

.eslintrc.js

@@ -24,7 +24,10 @@ module.exports = {
   rules: {
     '@typescript-eslint/naming-convention': 'off',
     '@typescript-eslint/no-use-before-define': 'off',
-    '@typescript-eslint/no-unused-vars': 'error',
+    '@typescript-eslint/no-unused-vars': [
+      'error',
+      { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
+    ],
     '@typescript-eslint/no-array-constructor': 'error',
     '@typescript-eslint/no-duplicate-imports': 'error',
     '@typescript-eslint/no-shadow': 'error',

.github/workflows/ci.yml

@@ -0,0 +1,40 @@
+name: CI
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    branches: [master]
+
+concurrency:
+  group: ci-${{ github.ref }}
+  cancel-in-progress: true
+
+# JavaScript side only. Building the apps needs Xcode and the Android SDK, so
+# that stays a local / release-pipeline step.
+jobs:
+  verify:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node: [20, 22]
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node }}
+          cache: pnpm
+
+      - run: pnpm install --frozen-lockfile
+
+      - name: Lint
+        run: pnpm run lint
+
+      - name: Type-check
+        run: pnpm run tslint
+
+      - name: Test
+        run: pnpm run test

CHANGELOG.md

@@ -0,0 +1,57 @@
+# Changelog
+
+## Unreleased
+
+This pass covers the JavaScript side only. See the note in the README about why
+React Native itself stays on 0.68.
+
+### Fixed
+
+- **`TextInput` imported a name that does not exist.** It pulled `deviceType`
+  from the `utils` barrel, which only re-exports it under its alias `isIOS`. At
+  runtime the import was `undefined`, so `if (!deviceType) return 14` always
+  took the first branch and the per-device font sizing never ran.
+- **Two environments had no API domain.** `Env` has eight members but
+  `apiServer` defined six — `local` and `bcp` resolved to `undefined`, making
+  the request base URL the string `"undefined"`. The map is now typed
+  `Record<Env, string>`, so a new environment cannot be added without one.
+- **The anti-double-submit `Button` never guarded anything.** It declared a
+  `loading` prop and tracked `iLoading` internally, and `setLoading(true)` ran
+  on press — but neither value was read anywhere, so repeat presses went
+  straight through. Presses during an in-flight `onPress` are now ignored.
+- **`Webview` could not be used as a navigation screen.** It declared its own
+  `route` shape, which is not assignable to React Navigation's
+  `ScreenComponentType`. It now uses `RouteProp` with typed params. The
+  `|| restProps` fallback next to it was dead: `restProps` only carries the
+  connected props, never `url` / `header` / `disableBottom`.
+- **`TabScreen` required a prop nothing supplies.** Its `connect()` call is
+  commented out, so `homeTodoCount` was always `undefined` and the badge always
+  read 0. The prop is now optional, which is what the code actually expects.
+- **The only test could never pass.** `__tests__/App-test.tsx` imported
+  `'../App'`, but `App.tsx` lives in `src/` — on a case-insensitive filesystem
+  that resolved to `app.json`, so React was handed an object.
+
+### Changed
+
+- TypeScript 4.5 → 5.9. 4.5 could not even parse the installed `@types/node`,
+  and `suppressImplicitAnyIndexErrors` (removed in TS 5.5) was hiding the
+  environment bug above. **0 type errors.**
+- ESLint config accepts `_`-prefixed unused bindings. **0 errors.**
+- Test stack realigned: it was jest 25 with babel-jest 28, `@types/jest` 27 and
+  react-test-renderer 16 against React 18. Now jest 29 throughout.
+- `transformIgnorePatterns` matches the React Native packages anywhere in the
+  path, because pnpm nests them under `node_modules/.pnpm/…` and the preset's
+  own pattern assumes a flat `node_modules`.
+- The whole-`<App />` render test is replaced by unit tests over the validation
+  helpers: 17 tests covering phone, bank card, ID card, URL and the password
+  rules. Rendering the full app needs the entire native module surface mocked
+  and does not finish in CI.
+- The project uses pnpm.
+
+### Added
+
+- English `README_EN.md` alongside the Chinese `readme.md`, both with badges and
+  a plain statement of the React Native version situation.
+- `CONTRIBUTING.md` and this changelog.
+- `ci.yml`: lint, type-check and test on Node 20 and 22. It deliberately does
+  not build the apps — that needs Xcode and the Android SDK.

CONTRIBUTING.md

@@ -0,0 +1,176 @@
+# Contributing · 贡献指南
+
+English | [中文](#中文)
+
+Thanks for taking the time to contribute. Issues and pull requests are both welcome.
+
+## "I don't have permission to push"
+
+You don't need it, and you shouldn't ask for it. **Nobody outside the project can push a branch to
+this repository** — that is how GitHub works for every public repo, not a restriction set up here.
+The way to contribute a change is to push to _your own_ fork and open a pull request from it:
+
+```bash
+# 1. Fork the repo on GitHub (the "Fork" button, top right)
+
+# 2. Clone YOUR fork, not this one
+git clone https://github.com/<your-username>/ReactNative-Production.git
+cd ReactNative-Production
+
+# 3. Point "upstream" at this repo so you can stay in sync
+git remote add upstream https://github.com/ludejun/ReactNative-Production.git
+
+# 4. Branch, commit, push to your fork
+git checkout -b fix/some-bug
+git commit -am "fix: describe what changed"
+git push origin fix/some-bug
+
+# 5. Open the pull request from your fork's branch against ludejun/master
+```
+
+Or do the whole thing with the [GitHub CLI](https://cli.github.com/):
+
+```bash
+gh repo fork ludejun/ReactNative-Production --clone
+cd ReactNative-Production
+git checkout -b fix/some-bug
+# ...edit, commit...
+gh pr create --repo ludejun/ReactNative-Production
+```
+
+A couple of things that can look like a permission problem but aren't:
+
+- **The checks on your PR sit there greyed out.** For a first-time contributor, GitHub Actions waits
+  for a maintainer to click "Approve and run". Nothing is wrong; it just needs a maintainer to look.
+- **`git push` to `ludejun/…` returns 403.** Expected — see above. Push to your fork's remote.
+
+## Development setup
+
+This project uses [pnpm](https://pnpm.io/) and needs Node >= 20. Building the apps additionally
+needs Xcode (iOS) or the Android SDK; the lint, type-check and test steps do not.
+
+```bash
+pnpm install
+pnpm start       # Metro
+```
+
+## Before you open the pull request
+
+Please make sure all four pass — CI runs exactly these:
+
+```bash
+pnpm lint        # eslint, must report 0 errors
+pnpm tslint      # tsc --noEmit
+pnpm test        # vitest
+pnpm test        # jest
+```
+
+If you changed behaviour, add or update a test in `tests/`. If you fixed a bug, a test that fails
+without your fix is the most useful thing you can include.
+
+Run `pnpm format` before committing so Prettier settles the formatting; CI does not reformat for you.
+
+## A few conventions
+
+- **Commit messages** follow [Conventional Commits](https://www.conventionalcommits.org/):
+  `fix:`, `feat:`, `docs:`, `chore:`, `refactor:`, `test:`.
+- **Both READMEs.** If a change affects how the framework is used, update `readme.md` _and_
+  `README_EN.md`.
+- **Keep the type check clean.** Several real bugs in this repo were hidden by
+  `suppressImplicitAnyIndexErrors` and an ancient TypeScript; `pnpm tslint` must stay at 0 errors.
+- **Test pure logic, not the whole app.** Rendering `<App />` needs the full native module surface
+  mocked and does not finish in CI. Put tests in `__tests__/*-test.ts`.
+
+## Reporting a bug
+
+Open an [issue](https://github.com/ludejun/ReactNative-Production/issues) with:
+
+- the React Native version, and whether it is iOS or Android,
+- the screen or component involved,
+- what you expected and what happened instead,
+- the error and stack trace, if there is one.
+
+---
+
+<a id="中文"></a>
+
+# 中文
+
+感谢你愿意花时间参与。Issue 和 Pull Request 都非常欢迎。
+
+## “我没有权限提交代码”
+
+你不需要这个权限，也不用来要。**项目之外的任何人都无法直接往本仓库推送分支** —— 这是 GitHub 对所有公开仓库的默认行为，不是本项目做了什么限制。正确的做法是推到**你自己的 fork**，再从 fork 发起 Pull Request：
+
+```bash
+# 1. 在 GitHub 页面右上角点 "Fork"
+
+# 2. clone 你自己的 fork，不是这个仓库
+git clone https://github.com/<你的用户名>/ReactNative-Production.git
+cd ReactNative-Production
+
+# 3. 把 upstream 指向本仓库，方便后续同步
+git remote add upstream https://github.com/ludejun/ReactNative-Production.git
+
+# 4. 建分支、提交、推到你自己的 fork
+git checkout -b fix/some-bug
+git commit -am "fix: 描述你改了什么"
+git push origin fix/some-bug
+
+# 5. 从你 fork 的这个分支，向 ludejun/master 发起 Pull Request
+```
+
+也可以用 [GitHub CLI](https://cli.github.com/) 一条龙：
+
+```bash
+gh repo fork ludejun/ReactNative-Production --clone
+cd ReactNative-Production
+git checkout -b fix/some-bug
+# ...改代码、提交...
+gh pr create --repo ludejun/ReactNative-Production
+```
+
+有两种情况看着像“没权限”，其实不是：
+
+- **PR 上的 CI 检查一直灰着不跑。** 首次贡献者的 workflow 需要维护者点一下 “Approve and run”，这是 GitHub 的默认策略，等一下即可。
+- **`git push` 到 `ludejun/…` 返回 403。** 这是预期行为，推到你自己 fork 的 remote 就好。
+
+## 本地开发
+
+本项目使用 [pnpm](https://pnpm.io/)，需要 Node >= 20。打包 App 另需 Xcode（iOS）或 Android SDK；lint、类型检查和测试不需要。
+
+```bash
+pnpm install
+pnpm start       # 启动 Metro
+```
+
+## 提 PR 之前
+
+请确认这四条全部通过 —— CI 跑的就是这四条：
+
+```bash
+pnpm lint        # eslint，必须 0 error
+pnpm tslint      # tsc --noEmit
+pnpm test        # vitest
+pnpm test        # jest
+```
+
+如果你改了行为，请在 `tests/` 下补充或更新测试。如果你修的是 bug，**一个不打补丁就会失败的测试**是最有价值的东西。
+
+提交前跑一下 `pnpm format` 让 Prettier 统一格式，CI 不会替你格式化。
+
+## 一些约定
+
+- **提交信息**遵循 [Conventional Commits](https://www.conventionalcommits.org/)：`fix:`、`feat:`、`docs:`、`chore:`、`refactor:`、`test:`。
+- **两份 README。** 如果改动影响了框架用法，请同时更新 `readme.md` 和 `README_EN.md`。
+- **保持类型检查干净。** 这个仓库里好几个真 bug 都是被 `suppressImplicitAnyIndexErrors` 和过旧的 TypeScript 盖住的，`pnpm tslint` 必须保持 0 error。
+- **测纯逻辑，不要测整个 App。** 渲染 `<App />` 需要 mock 全部原生模块，在 CI 里跑不完。测试放在 `__tests__/*-test.ts`。
+
+## 反馈 Bug
+
+到 [Issues](https://github.com/ludejun/ReactNative-Production/issues) 提一条，请带上：
+
+- React Native 版本，以及是 iOS 还是 Android，
+- 涉及哪个页面或组件，
+- 你期望的行为，以及实际发生了什么，
+- 如果有报错，附上错误信息和堆栈。