Update project metadata and CI workflow - #51

Merged
sis0k0 merged 8 commits into
mainfrom
update-lf
May 27, 2026
Merged

Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom
update-lf

Conversation

@sis0k0

@sis0k0sis0k0 commented May 18, 2026•
edited
Loading

Copy link
Copy Markdown
Member

This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.

Documentation

  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture

Developer environment

  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:
    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration

Database tooling

  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start

CI

  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions

Testing

CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.

@sis0k0
sis0k0 requested a review from CopilotMay 18, 2026 14:55
CopilotAI reviewed May 18, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.

Changes:

  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md Outdated
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml Outdated
Comment on lines +39 to +44
- name: Start server in the background
env:
env:
ATLAS_URI: ${{ secrets.ATLAS_URI }}
run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
run: |
echo "ATLAS_URI=$ATLAS_URI" > config.env
npm start &
@sis0k0
sis0k0 requested a review from CopilotMay 19, 2026 14:11
CopilotAI reviewed May 19, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js Outdated
Comment threadmern/server/seed.js
Comment threadREADME.md
Comment on lines +54 to 59
### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js Outdated
Comment threadmern/client/cypress/e2e/endToEnd.cy.js Outdated
Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json Outdated
@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This was referenced May 22, 2026
Merged
Merged
Merged
Merged
Merged
Merged
@AfiMaameDufie

AfiMaameDufie commented May 22, 2026

Copy link
Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.

@sis0k0
sis0k0 merged commit 75a3989 into mainMay 27, 2026
3 checks passed
@sis0k0
sis0k0 deleted the update-lf branch May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
Update project metadata and CI workflow - #51
Merged
sis0k0 merged 8 commits into
mainfrom 
update-lf
May 27, 2026
Merged
Update project metadata and CI workflow#51
sis0k0 merged 8 commits into
mainfrom 
update-lf
Conversation
@sis0k0
@sis0k0sis0k0
commented
May 18, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR improves the developer experience and CI reliability for the MERN stack example through several related changes.
Documentation
  • Expanded README with architecture overview, quick start guide, Node.js version requirement (20+, required for node --env-file), and MongoDB feature links
  • Added Codespaces badge for one-click environment setup
  • Added AGENTS.md with contributor and agent guidance
  • Added EDD.md (Engineering Design Document) describing the data model and architecture
Developer environment
  • Added .devcontainer/ with full GitHub Codespaces and VS Code Dev Container support:

    • devcontainer.json — Node 22, VS Code extensions (MongoDB, TypeScript, Prettier), port forwarding
    • docker-compose.yml — MongoDB Atlas Local service with persistent volumes and health checks
    • README.md — setup guide with troubleshooting
  • Added mern/server/config.env.example — template for local configuration
Database tooling
  • Added mern/server/seed.js — populates employees.records with sample data; used automatically in the devcontainer on start
CI
  • Switched to mongo:latest service container so CI runs without an Atlas secret (works on fork PRs)
  • Added ATLAS_URI written directly from the service container credentials — no secrets required
  • Added server readiness check (wait-on) before Cypress runs to prevent race conditions
  • Migrated Cypress from v9 (cypress.json + integration/) to v10+ (cypress.config.js + e2e/)
  • Fixed test assertions to match current UI text
  • Bumped Node matrix to 20.x and 22.x; updated actions to latest versions
Testing
CI runs on both Node 20 and 22 against a local MongoDB service container. The devcontainer setup has been manually verified with MongoDB Atlas Local.
@sis0k0
sis0k0
requested a review
from CopilotMay 18, 2026 14:55
CopilotAI
reviewed
May 18, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Updates repository documentation and automation to better describe the MERN employee records app, adds project metadata docs, introduces a database seed script, and modernizes the GitHub Actions CI workflow.
Changes:
  • Expanded README with architecture, quick start, and MongoDB feature links.
  • Added seed.js for populating employees.records, plus new repo docs (AGENTS.md, EDD.md) and a Claude plugin settings file.
  • Updated CI workflow to use newer Node versions, newer actions, and run Cypress via cypress-io/github-action.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
FileDescription
README.mdReplaces minimal README with a full overview, architecture diagram, and setup instructions.
mern/server/seed.jsAdds a MongoDB seed script for sample employee records.
EDD.mdDocuments the data model and includes a Mermaid ER diagram.
AGENTS.mdAdds an AI agent guide describing structure and common commands.
.github/workflows/main.yamlModernizes CI (Node matrix, actions versions, Cypress action usage).
.claude/settings.jsonEnables the official MongoDB Claude plugin for this repo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment threadREADME.md
Comment threadmern/server/seed.js
Comment threadmern/server/seed.js
Comment threadEDD.md
Outdated

Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Outdated


Comment on lines

+39
 to 
+44

 - name: Start server in the background
 env:
 env:
 ATLAS_URI: ${{ secrets.ATLAS_URI }}
 run: (cd mern/server && echo "ATLAS_URI=$ATLAS_URI" > config.env && npm start &)

- name: Start React app in the background
run: (cd mern/client && npm start &)
 run: |
 echo "ATLAS_URI=$ATLAS_URI" > config.env
 npm start &
@sis0k0
sis0k0
requested a review
from CopilotMay 19, 2026 14:11
CopilotAI
reviewed
May 19, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.
Comment thread.github/workflows/main.yaml
Comment thread.github/workflows/main.yaml
Comment threadmern/server/seed.js
Outdated

Comment threadmern/server/seed.js
Comment threadREADME.md

Comment on lines

+54
 to 
59

### 2. Seed the database (optional)

```bash
cd mern/server
node seed.js
```
Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadmern/client/cypress/e2e/endToEnd.cy.js
Outdated

Comment threadEDD.md
Comment thread.devcontainer/docker-compose.yml
Comment thread.devcontainer/devcontainer.json
Outdated

@sis0k0
Apply suggestions from code review
fc3f108
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This was referenced May 22, 2026
 Merged

 Merged

 Merged

 Merged

 Merged

 Merged

@AfiMaameDufie
AfiMaameDufie
commented
May 22, 2026

Copy link


Copy Markdown

I went through this and the files look good to me. It's mostly adding more details, tests and diagrams to the processes.
@sis0k0
sis0k0
merged commit 75a3989
into
mainMay 27, 2026

3 checks passed

@sis0k0
sis0k0
deleted the

update-lf

branch
May 27, 2026 08:05
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@sis0k0@AfiMaameDufie