mirror of
https://github.com/PR0M3TH3AN/bitvid.git
synced 2026-10-27 07:52:24 +00:00
docs(ai): improve onboarding steps
Audited the onboarding process and confirmed all steps pass. Added .npmrc with engine-strict=true to enforce Node.js >=22 requirement. Updated CONTRIBUTING.md and README.md to explicitly mention the Node version requirement and its enforcement. Added artifacts/ONBOARDING_REPORT.md with the audit report. Co-authored-by: PR0M3TH3AN <[email protected]>
This commit is contained in:
1 parent
58412220fc
commit
13c626b5b6
4 files changed
+32
-2
No files matched your search
+1
-1
@@ -93,7 +93,7 @@ This workflow uses the `force-cancel` endpoint to ensure stalled runs are termin
|
|||||||
To set up the project locally:
|
To set up the project locally:
|
||||||
|
|
||||||
1. **Prerequisites**:
|
1. **Prerequisites**:
|
||||||
- **Node.js**: v22 or higher.
|
- **Node.js**: v22 or higher (enforced by `.npmrc`).
|
||||||
- **NPM**: v10 or higher (included with Node 22).
|
- **NPM**: v10 or higher (included with Node 22).
|
||||||
|
|
||||||
2. **Install Dependencies**:
|
2. **Install Dependencies**:
|
||||||
|
|||||||
@@ -143,7 +143,7 @@ To run **bitvid** locally:
|
|||||||
|
|
||||||
2. Install dependencies:
|
2. Install dependencies:
|
||||||
|
|
||||||
Use `npm ci` to install dependencies exactly as specified in `package-lock.json`. This ensures `npx` finds the correct local `tailwindcss` version.
|
The project requires Node.js v22 or higher. Use `npm ci` to install dependencies exactly as specified in `package-lock.json`.
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
npm ci
|
npm ci
|
||||||
|
|||||||
@@ -0,0 +1,29 @@
|
|||||||
|
# Onboarding Audit Report
|
||||||
|
Date: 2025-02-11
|
||||||
|
|
||||||
|
## Summary
|
||||||
|
The onboarding process was audited by simulating a fresh checkout and running the documented commands. All steps passed successfully. To further improve reliability, strict Node version enforcement was added.
|
||||||
|
|
||||||
|
## Environment
|
||||||
|
- **Node Version**: v22.22.0
|
||||||
|
- **NPM Version**: 11.7.0
|
||||||
|
- **OS**: Linux (Sandbox)
|
||||||
|
|
||||||
|
## Commands Executed
|
||||||
|
1. `npm ci` - **PASS**
|
||||||
|
- Clean install of dependencies.
|
||||||
|
- Added 424 packages.
|
||||||
|
2. `npm run build:css` - **PASS**
|
||||||
|
- Built Tailwind CSS successfully.
|
||||||
|
3. `npm run format` - **PASS**
|
||||||
|
- Checked formatting for CSS, HTML, MD, Config.
|
||||||
|
- No files were modified (compliance verified).
|
||||||
|
|
||||||
|
## Configuration Checks
|
||||||
|
- **package.json**: `engines.node` is set to `>=22`.
|
||||||
|
- **.devcontainer/devcontainer.json**: Uses `mcr.microsoft.com/devcontainers/javascript-node:22`, consistent with `package.json`.
|
||||||
|
- **Dockerfile**: Uses `mcr.microsoft.com/playwright:v1.58.0-jammy`. Verified compatible with Node 22 requirement.
|
||||||
|
|
||||||
|
## Improvements Implemented
|
||||||
|
- **Enforce Node Version**: created `.npmrc` with `engine-strict=true`. This ensures developers are using the required Node version (>=22) during `npm install`/`ci`.
|
||||||
|
- **Documentation**: Updated `README.md` and `CONTRIBUTING.md` to explicitly mention the Node version requirement and its enforcement.
|
||||||
Reference in new issue
Block a user