-
Notifications
You must be signed in to change notification settings - Fork 13
docs: improve setup documentation and add .env.sample template #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,17 @@ | ||||||
| # Alpha One Labs Education Platform - Environment Variables Template | ||||||
| # Copy this file to `.dev.vars` for local development. | ||||||
| # Do not commit real secrets to version control. | ||||||
|
|
||||||
| # ======================================== | ||||||
| # Encryption | ||||||
| # ======================================== | ||||||
| # Used to encrypt user PII (Personal Identifiable Information) and activity data | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix PII terminology on Line 8. Use “Personally Identifiable Information” (not “Personal Identifiable Information”) for correctness in security/privacy docs. Suggested patch-# Used to encrypt user PII (Personal Identifiable Information) and activity data
+# Used to encrypt user PII (Personally Identifiable Information) and activity data📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| # before storing it in the database. Ensure this is a strong randomly generated string. | ||||||
| ENCRYPTION_KEY=your-dev-encryption-key-here | ||||||
|
|
||||||
| # ======================================== | ||||||
| # Authentication | ||||||
| # ======================================== | ||||||
| # Used to sign JSON Web Tokens (JWT) for stateless user authentication. | ||||||
| # Keep this secret to prevent unauthorized token forging. | ||||||
| JWT_SECRET=your-dev-jwt-secret-here | ||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -14,102 +14,113 @@ Alpha One Labs is an education platform designed to facilitate both learning and | |||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| ### Prerequisites | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Make sure you have installed: | ||||||||||||||||||||||||||||||||||||||||||||||
| - Node.js | ||||||||||||||||||||||||||||||||||||||||||||||
| - Wrangler CLI | ||||||||||||||||||||||||||||||||||||||||||||||
| Before starting, ensure you have the following installed: | ||||||||||||||||||||||||||||||||||||||||||||||
| - **Node.js**: v18.0.0 or higher | ||||||||||||||||||||||||||||||||||||||||||||||
| - **npm**: v9.0.0 or higher | ||||||||||||||||||||||||||||||||||||||||||||||
| - **Wrangler CLI**: For deploying Cloudflare Workers | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Install Wrangler: | ||||||||||||||||||||||||||||||||||||||||||||||
| Install Wrangler globally using npm: | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||||
| npm install -g wrangler | ||||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| ### Clone the Repository | ||||||||||||||||||||||||||||||||||||||||||||||
| ### Fork & Clone the Repository (For Contributors) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| If you are a contributor, first fork the repository to your own GitHub account. | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||||
| git clone https://github.com/alphaonelabs/learn.git | ||||||||||||||||||||||||||||||||||||||||||||||
| # Clone your fork (replace <your-username> with your actual username) | ||||||||||||||||||||||||||||||||||||||||||||||
| git clone https://github.com/<your-username>/learn.git | ||||||||||||||||||||||||||||||||||||||||||||||
| cd learn | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| # Add the original repository as an upstream remote | ||||||||||||||||||||||||||||||||||||||||||||||
| git remote add upstream https://github.com/alphaonelabs/learn.git | ||||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| ### Login to Cloudflare (One time) | ||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||
| ### Login to Cloudflare (One time) | |
| ### Login to Cloudflare (One-time) |
Copilot
AI
Mar 30, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The D1 setup instructions refer to updating a placeholder database_id, but the repo’s wrangler.toml currently has a concrete UUID committed. This is confusing for new contributors (they may not have access to that D1 DB). Consider adjusting the text to explain they should replace the existing database_id with the UUID from their own wrangler d1 create output (or provide a separate dev/prod config approach).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix ordered-list numbering to satisfy markdownlint (MD029).
This repo’s lint rule expects 1. for each ordered item, not 1/2/3.
Lint-safe numbering
-2. **Add configuration to `wrangler.toml`:**
+1. **Add configuration to `wrangler.toml`:**
@@
-3. **Apply the Schema:**
+1. **Apply the Schema:**📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| 1. **Create the Database:** | |
| ```bash | |
| wrangler d1 create education_db | |
| ``` | |
| - Add the generated database_id to your wrangler.toml: | |
| 2. **Add configuration to `wrangler.toml`:** | |
| After running the create command, copy the `database_id` from the output and update your `wrangler.toml` file with this placeholder: | |
| ```toml | |
| [[d1_databases]] | |
| binding = "DB" | |
| database_name = "education_db" | |
| database_id = "YOUR_DATABASE_ID" | |
| database_id = "<YOUR_DATABASE_ID>" | |
| ``` | |
| - Apply Schema: | |
| 3. **Apply the Schema:** | |
| 1. **Create the Database:** | |
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 59-59: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1
(MD029, ol-prefix)
[warning] 69-69: Ordered list item prefix
Expected: 1; Actual: 3; Style: 1/1/1
(MD029, ol-prefix)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 53 - 69, The ordered list numbering in the README
segment containing "Create the Database:", "Add configuration to
`wrangler.toml`:", and "Apply the Schema:" must use repeated "1." entries to
satisfy markdownlint MD029; update each numeric prefix (currently 1/2/3) so
every list item begins with "1." while preserving the existing content and
fenced code blocks (the wrangler d1 create command and toml snippet) and
spacing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a language to fenced code blocks (MD040).
The fence should include a language tag for lint compliance.
Minimal fix
-```
+```env
ENCRYPTION_KEY=your-dev-encryption-key
JWT_SECRET=your-dev-jwt-secret</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 84-84: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 84 - 87, The fenced code block showing environment
variables in README.md lacks a language tag; update the block delimiter from ```
to include a language (e.g., use ```env) so the snippet becomes a fenced code
block with a language tag for lint MD040 compliance, keeping the same lines
"ENCRYPTION_KEY=your-dev-encryption-key" and "JWT_SECRET=your-dev-jwt-secret".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wording tweak: use “separate terminals” instead of “separate instructions.”
This improves execution clarity for new contributors.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 100, Replace the phrase "separate instructions" with
"separate terminals" in the README sentence describing how to run the backend
worker and frontend HTML files so it reads that they should be run concurrently
in separate terminals; update the exact sentence where the README currently says
"Run them concurrently in separate instructions." to "Run them concurrently in
separate terminals." to improve clarity for new contributors.
Copilot
AI
Mar 30, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
“Run them concurrently in separate instructions” reads like a typo/wording error (likely meant “separate terminals” or “separate shells”). Tweaking this phrasing would make the setup steps clearer.
| The application consists of a backend worker and frontend HTML files. Run them concurrently in separate instructions. | |
| The application consists of a backend worker and frontend HTML files. Run them concurrently in separate terminal sessions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a trailing newline at end of file (MD047).
README should end with exactly one newline to pass lint.
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 126-126: Files should end with a single newline character
(MD047, single-trailing-newline)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 126, The README.md currently lacks the required trailing
newline (MD047); open README.md and ensure the file ends with exactly one
newline character (no extra blank lines or missing newline) so the final byte is
a single newline to satisfy the linter rule MD047.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PII is typically expanded as “Personally Identifiable Information”, not “Personal Identifiable Information”. Update the wording to the standard expansion.