fix: devcontainer seed timing, port conflict, and Codespaces mixed content

- Wait for Atlas Local replica set primary election (isWritablePrimary)
  instead of ping before seeding; up to 60 retries × 2s
- Remove duplicate server/client start from startup.sh — postAttachCommand
  already handles those, double-start caused port 5050 conflict
- Add Vite proxy for /record → http://localhost:5050 so browser fetches
  go same-origin (fixes HTTPS mixed-content blocking in Codespaces)
- Replace all hardcoded http://localhost:5050/record URLs in React
  components with relative /record paths

Author: sis0k0

.devcontainer/devcontainer.json

@@ -33,7 +33,7 @@
 
   // this will open 3 bash sessions, each one running the specified command
   "postAttachCommand": {
-    "server": "cd mern/server && npm start",
+    "server": "cd mern/server && node ./seed.js && npm start",
     "client": "cd mern/client && npm run dev -- --host 0.0.0.0",
     "portSetup": "sleep 5 && .devcontainer/portSetup.sh"
   },

.devcontainer/startup.sh

@@ -18,45 +18,28 @@ EOF
 }
 
 wait_for_mongodb() {
-  local max_tries=30
+  local max_tries=60
   local try=1
 
-  until node --input-type=module -e "import { MongoClient } from 'mongodb'; const c = new MongoClient(process.env.ATLAS_URI); await c.connect(); await c.db('admin').command({ ping: 1 }); await c.close();" >/dev/null 2>&1; do
+  # Atlas Local uses a replica set — wait until a writable primary is elected,
+  # not just until mongod responds to ping (which happens before primary election).
+  until node --input-type=module -e "
+    import { MongoClient } from 'mongodb';
+    const c = new MongoClient(process.env.ATLAS_URI);
+    await c.connect();
+    const h = await c.db('admin').command({ hello: 1 });
+    await c.close();
+    if (!h.isWritablePrimary) throw new Error('no primary yet');
+  " >/dev/null 2>&1; do
     if (( try >= max_tries )); then
-      echo "MongoDB did not become ready in time."
+      echo "MongoDB primary did not become ready in time."
       return 1
     fi
-    echo "Waiting for MongoDB... ($try/$max_tries)"
+    echo "Waiting for MongoDB primary... ($try/$max_tries)"
     try=$((try + 1))
     sleep 2
   done
-}
-
-seed_database() {
-  (
-    cd "$SERVER_DIR"
-    node --env-file=config.env seed.js > /tmp/mern-seed.log 2>&1
-  )
-
-  echo "Seed completed (log: /tmp/mern-seed.log)."
-}
-
-verify_seed_data() {
-  local count
-
-  count=$(
-    cd "$SERVER_DIR" &&
-      node --env-file=config.env --input-type=module -e "import { MongoClient } from 'mongodb'; const c = new MongoClient(process.env.ATLAS_URI); await c.connect(); const n = await c.db('employees').collection('records').countDocuments(); console.log(n); await c.close();"
-  )
-
-  echo "Seed verification: employees.records has $count documents."
-
-  if [[ "$count" -le 0 ]]; then
-    echo "Seed verification failed: no documents found in employees.records."
-    echo "Seed log tail:"
-    tail -n 50 /tmp/mern-seed.log || true
-    return 1
-  fi
+  echo "MongoDB primary is ready."
 }
 
 start_server_if_needed() {
@@ -87,9 +70,5 @@ start_client_if_needed() {
 
 ensure_config
 wait_for_mongodb
-seed_database
-verify_seed_data
-start_server_if_needed
-start_client_if_needed
 
 echo "Codespaces startup complete."

mern/client/src/components/Record.jsx

@@ -26,7 +26,7 @@ export default function Record() {
       if(!id) return;
       setIsNew(false);
       const response = await fetch(
-        `http://localhost:5050/record/${params.id.toString()}`
+        `/record/${params.id.toString()}`
       );
       if (!response.ok) {
         const message = `An error has occurred: ${response.statusText}`;
@@ -60,7 +60,7 @@ export default function Record() {
       let response;
       if (isNew) {
         // if we are adding a new record we will POST to /record.
-        response = await fetch("http://localhost:5050/record", {
+        response = await fetch("/record", {
           method: "POST",
           headers: {
             "Content-Type": "application/json",
@@ -69,7 +69,7 @@ export default function Record() {
         });
       } else {
         // if we are updating a record we will PATCH to /record/:id.
-        response = await fetch(`http://localhost:5050/record/${params.id}`, {
+        response = await fetch(`/record/${params.id}`, {
           method: "PATCH",
           headers: {
             "Content-Type": "application/json",

mern/client/src/components/RecordList.jsx

@@ -50,7 +50,7 @@ export default function RecordList() {
   // This method fetches the records from the database.
   useEffect(() => {
     async function getRecords() {
-      const response = await fetch(`http://localhost:5050/record/`);
+      const response = await fetch(`/record/`);
       if (!response.ok) {
         const message = `An error occurred: ${response.statusText}`;
         console.error(message);
@@ -65,7 +65,7 @@ export default function RecordList() {
 
   // This method will delete a record
   async function deleteRecord(id) {
-    await fetch(`http://localhost:5050/record/${id}`, {
+    await fetch(`/record/${id}`, {
       method: "DELETE",
     });
     const newRecords = records.filter((el) => el._id !== id);

mern/client/vite.config.js

@@ -4,4 +4,9 @@ import react from '@vitejs/plugin-react'
 // https://vitejs.dev/config/
 export default defineConfig({
   plugins: [react()],
+  server: {
+    proxy: {
+      '/record': 'http://localhost:5050',
+    },
+  },
 })