remove smoke tests

Author: nicolethoen

BOOKMARKS.md

@@ -45,6 +45,18 @@ Why the library lives under `packages/module/` instead of a root `src/` / `tests
 
 **Added by**: AgentReady remediation | **Date**: 2026-04-09
 
+### [ADR 0002 — Jest at the workspace root](./docs/adr/0002-unit-testing-with-jest.md)
+
+Jest config at the repo root, `yarn test` / `--passWithNoTests`, and lint-staged related tests.
+
+**Added by**: AgentReady remediation | **Date**: 2026-04-10
+
+### [ADR 0003 — CI PR checks (reusable workflow)](./docs/adr/0003-ci-pr-checks-via-reusable-workflow.md)
+
+PR checks via `check-pr` calling the reusable `build-lint-test` workflow.
+
+**Added by**: AgentReady remediation | **Date**: 2026-04-10
+
 ---
 
 ## Development Resources

CLAUDE.md

@@ -1,6 +1,6 @@
 # react-user-feedback
 
-A React component library for collecting user feedback via a form in a modal. Built with React, TypeScript, PatternFly, and Webpack.
+A React component library for collecting user feedback via a form in a modal. Built with React, TypeScript, and PatternFly; the library builds with **TypeScript (`tsc`)**, and local extension docs use the **PatternFly documentation framework** (`pf-docs-framework`).
 
 ## Structure
 
@@ -25,9 +25,9 @@ This repo is a **Yarn workspaces** monorepo. The **shipped library** is **`@patt
 
 ```bash
 yarn build          # Build the library
-yarn test           # Run Jest for packages/
+yarn test           # Run Jest for packages/ (passes with no tests until you add *.test.* / __tests__; uses --passWithNoTests)
 yarn lint           # Lint all of packages/ (like CI)
-yarn lint-staged    # What Husky runs on commit: Prettier + ESLint + jest --findRelatedTests on staged files only
+yarn lint-staged    # What Husky runs on commit: Prettier + ESLint + jest --findRelatedTests --passWithNoTests on staged files only
 ```
 
 For accessibility testing:

README.md

@@ -17,7 +17,7 @@ yarn build
 
 **Lint:** **`yarn lint`** — ESLint for JS/TS and Markdown under `packages/` (see `package.json` for `lint:js` / `lint:md` / `lint:markdownlint`).
 
-**Unit tests:** Jest uses `jest.setup.js`; tests live under `packages/`. Run **`yarn test`**. **Husky** runs **`lint-staged`** on commit; on staged `packages/**/*.{js,jsx,ts,tsx}` it runs Prettier, ESLint, and **`jest --findRelatedTests`**.
+**Unit tests:** Jest uses `jest.setup.js` at the repo root; add tests under `packages/` (e.g. `*.test.*` or `__tests__/`). Run **`yarn test`** — it uses **`--passWithNoTests`** so CI stays green until tests exist. **Husky** runs **`lint-staged`** on commit; on staged `packages/**/*.{js,jsx,ts,tsx}` it runs Prettier, ESLint, and **`jest --findRelatedTests`** (with **`--passWithNoTests`**).
 
 **Accessibility:** use two terminals — (1) `yarn build:docs && yarn serve:docs` (2) `yarn test:a11y`. **`yarn serve:a11y`** opens the generated a11y report after tests.
 

docs/adr/0002-unit-testing-with-jest.md

@@ -8,13 +8,13 @@ Accepted
 
 ## Context
 
-The library ships from **`packages/module`**, but contributors expect a single **`yarn test`** entry point from the repository root. We need consistent time zones for snapshots, a minimal guard against an empty Jest configuration, and fast feedback on pre-commit for staged changes.
+The library ships from **`packages/module`**, but contributors expect a single **`yarn test`** entry point from the repository root. We need consistent time zones for snapshots, and fast feedback on pre-commit for staged changes. The root **`yarn test`** uses **`--passWithNoTests`** so CI and local runs stay green until real component tests are added.
 
 ## Decision
 
 - **Jest is configured at the repo root** (`jest.config.js`, `jest.setup.js`) with **`roots`** aimed at **`packages/`** so tests in the workspace are discovered without a root-level `tests/` directory.
 - **Default timezone** for tests is set in **`jest.setup.js`** (and mirrored in the **`yarn test`** script where needed) so date-sensitive output is stable in CI and locally.
-- A small **smoke test** under `packages/module` ensures the suite is non-empty and the wiring stays valid.
+- **`yarn test`** passes with **no test files** (`--passWithNoTests`) so we do not maintain placeholder tests solely to satisfy Jest.
 - **lint-staged** may run **`jest --findRelatedTests`** on staged source files to keep pre-commit fast.
 
 ## Consequences

package.json

@@ -19,7 +19,7 @@
     "lint:markdownlint": "markdownlint --dot -p .markdownlintignore \"**/*.md\"",
     "lint": "yarn lint:js && yarn lint:md && yarn lint:markdownlint",
     "lint-staged": "lint-staged",
-    "test": "TZ=EST jest packages",
+    "test": "TZ=EST jest packages --passWithNoTests",
     "test:a11y": "yarn workspace @patternfly/react-user-feedback test:a11y",
     "serve:a11y": "yarn workspace @patternfly/react-user-feedback serve:a11y",
     "prepare": "husky install"