Compare commits
35 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| cd16b7ea28 | |||
| 20d8a13375 | |||
| f72aaa52f8 | |||
| dd47d5006e | |||
| 10e25e1803 | |||
| afbdefa549 | |||
| 1a62e2fdd9 | |||
| 1271c061fd | |||
| 7b2a996c21 | |||
| 3233d65db0 | |||
| f1e192c5d4 | |||
| 3037381084 | |||
| e54c1d4848 | |||
| 3ae3f98df5 | |||
| 65d6514603 | |||
| bc44bcbde9 | |||
| cae0f2222a | |||
| 28833a7ec6 | |||
| 6ba02bf17d | |||
| bfe71b2511 | |||
| 0f2a37cb39 | |||
| 73f4eabbc7 | |||
| 515ff8ddb3 | |||
| 08c12c9394 | |||
| 4ce7df4649 | |||
| 6c04a30c3a | |||
| c6cd8098fd | |||
| 15ed329743 | |||
| 1412b2e0b7 | |||
| 30b037ff9c | |||
| 7a5b5d7afc | |||
| 3383bee968 | |||
| 0c30e4bd29 | |||
| 01f83d25f6 | |||
| 79adc365d8 |
@@ -2,7 +2,8 @@
|
|||||||
"permissions": {
|
"permissions": {
|
||||||
"allow": [
|
"allow": [
|
||||||
"Bash(npm test:*)",
|
"Bash(npm test:*)",
|
||||||
"Bash(npm install:*)"
|
"Bash(npm install:*)",
|
||||||
|
"Bash(find /c/Users/josh1/Documents/Code/Catalyst -type f \\\\\\(-name *.test.js -o -name *.spec.js -o -name .env* -o -name *.config.js \\\\\\))"
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,84 +0,0 @@
|
|||||||
name: Build
|
|
||||||
|
|
||||||
on:
|
|
||||||
push:
|
|
||||||
branches: [main]
|
|
||||||
tags:
|
|
||||||
- 'v*'
|
|
||||||
|
|
||||||
env:
|
|
||||||
IMAGE: ${{ vars.REGISTRY_HOST }}/${{ gitea.repository_owner }}/catalyst
|
|
||||||
|
|
||||||
jobs:
|
|
||||||
test:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
|
|
||||||
steps:
|
|
||||||
- name: Checkout
|
|
||||||
uses: actions/checkout@v4
|
|
||||||
|
|
||||||
- name: Setup Node
|
|
||||||
uses: actions/setup-node@v4
|
|
||||||
with:
|
|
||||||
node-version: 'lts/*'
|
|
||||||
cache: npm
|
|
||||||
|
|
||||||
- name: Install dependencies
|
|
||||||
run: npm ci
|
|
||||||
|
|
||||||
- name: Run tests
|
|
||||||
run: npm test
|
|
||||||
|
|
||||||
build:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
needs: test
|
|
||||||
if: startsWith(gitea.ref, 'refs/tags/v')
|
|
||||||
|
|
||||||
steps:
|
|
||||||
- name: Checkout
|
|
||||||
uses: actions/checkout@v4
|
|
||||||
|
|
||||||
- name: Docker metadata
|
|
||||||
id: meta
|
|
||||||
uses: docker/metadata-action@v5
|
|
||||||
with:
|
|
||||||
images: ${{ env.IMAGE }}
|
|
||||||
tags: |
|
|
||||||
type=semver,pattern={{version}}
|
|
||||||
type=semver,pattern={{major}}.{{minor}}
|
|
||||||
type=sha,prefix=,format=short
|
|
||||||
type=raw,value=latest,enable={{is_default_branch}}
|
|
||||||
|
|
||||||
- name: Log in to Gitea registry
|
|
||||||
uses: docker/login-action@v3
|
|
||||||
with:
|
|
||||||
registry: ${{ vars.REGISTRY_HOST }}
|
|
||||||
username: ${{ gitea.actor }}
|
|
||||||
password: ${{ secrets.TOKEN }}
|
|
||||||
|
|
||||||
- name: Build and push
|
|
||||||
uses: docker/build-push-action@v5
|
|
||||||
with:
|
|
||||||
context: .
|
|
||||||
push: true
|
|
||||||
tags: ${{ steps.meta.outputs.tags }}
|
|
||||||
|
|
||||||
release:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
needs: build
|
|
||||||
if: startsWith(gitea.ref, 'refs/tags/v')
|
|
||||||
|
|
||||||
steps:
|
|
||||||
- name: Create release
|
|
||||||
run: |
|
|
||||||
curl -sf -X POST \
|
|
||||||
-H "Authorization: token ${{ secrets.TOKEN }}" \
|
|
||||||
-H "Content-Type: application/json" \
|
|
||||||
"${{ gitea.server_url }}/api/v1/repos/${{ gitea.repository }}/releases" \
|
|
||||||
-d "{
|
|
||||||
\"tag_name\": \"${{ gitea.ref_name }}\",
|
|
||||||
\"name\": \"Catalyst ${{ gitea.ref_name }}\",
|
|
||||||
\"body\": \"### Image\n\n\`${{ env.IMAGE }}:${{ gitea.ref_name }}\`\",
|
|
||||||
\"draft\": false,
|
|
||||||
\"prerelease\": false
|
|
||||||
}"
|
|
||||||
53
.gitea/workflows/ci.yml
Normal file
53
.gitea/workflows/ci.yml
Normal file
@@ -0,0 +1,53 @@
|
|||||||
|
name: CI
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [dev, main]
|
||||||
|
pull_request:
|
||||||
|
branches: [dev, main]
|
||||||
|
|
||||||
|
env:
|
||||||
|
IMAGE: ${{ vars.REGISTRY_HOST }}/${{ gitea.repository_owner }}/catalyst
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
test:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version: 'lts/*'
|
||||||
|
|
||||||
|
- run: npm ci
|
||||||
|
|
||||||
|
- run: npm test
|
||||||
|
|
||||||
|
build-dev:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
needs: test
|
||||||
|
if: github.event_name == 'push' && github.ref == 'refs/heads/dev'
|
||||||
|
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Log in to registry
|
||||||
|
uses: docker/login-action@v3
|
||||||
|
with:
|
||||||
|
registry: ${{ vars.REGISTRY_HOST }}
|
||||||
|
username: ${{ gitea.actor }}
|
||||||
|
password: ${{ secrets.TOKEN }}
|
||||||
|
|
||||||
|
- name: Compute short SHA
|
||||||
|
run: echo "SHORT_SHA=$(git rev-parse --short HEAD)" >> $GITEA_ENV
|
||||||
|
|
||||||
|
- name: Build and push
|
||||||
|
uses: docker/build-push-action@v5
|
||||||
|
with:
|
||||||
|
context: .
|
||||||
|
push: true
|
||||||
|
build-args: BUILD_VERSION=dev-${{ env.SHORT_SHA }}
|
||||||
|
tags: |
|
||||||
|
${{ env.IMAGE }}:dev
|
||||||
|
${{ env.IMAGE }}:dev-${{ gitea.sha }}
|
||||||
87
.gitea/workflows/release.yml
Normal file
87
.gitea/workflows/release.yml
Normal file
@@ -0,0 +1,87 @@
|
|||||||
|
name: Release
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
|
||||||
|
env:
|
||||||
|
IMAGE: ${{ vars.REGISTRY_HOST }}/${{ gitea.repository_owner }}/catalyst
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
release:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
with:
|
||||||
|
fetch-depth: 0
|
||||||
|
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version: 'lts/*'
|
||||||
|
|
||||||
|
- run: npm ci
|
||||||
|
- run: npm test
|
||||||
|
|
||||||
|
- name: Read version
|
||||||
|
run: |
|
||||||
|
VERSION=$(node -p "require('./package.json').version")
|
||||||
|
echo "VERSION=${VERSION}" >> $GITEA_ENV
|
||||||
|
|
||||||
|
- name: Assert tag does not exist
|
||||||
|
run: |
|
||||||
|
if git ls-remote --tags origin "refs/tags/v${{ env.VERSION }}" | grep -q .; then
|
||||||
|
echo "ERROR: tag v${{ env.VERSION }} already exists — bump version in package.json before merging to main."
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
- name: Create and push tag
|
||||||
|
run: |
|
||||||
|
git config user.name "gitea-actions"
|
||||||
|
git config user.email "actions@gitea"
|
||||||
|
git tag "v${{ env.VERSION }}"
|
||||||
|
git push origin "v${{ env.VERSION }}"
|
||||||
|
|
||||||
|
- name: Generate release notes
|
||||||
|
run: |
|
||||||
|
LAST_TAG=$(git describe --tags --abbrev=0 HEAD^ 2>/dev/null || echo "")
|
||||||
|
if [ -n "$LAST_TAG" ]; then
|
||||||
|
git log "${LAST_TAG}..HEAD" --pretty=format:"- %s" --no-merges > /tmp/release_notes.txt
|
||||||
|
else
|
||||||
|
git log --pretty=format:"- %s" --no-merges > /tmp/release_notes.txt
|
||||||
|
fi
|
||||||
|
|
||||||
|
- name: Docker metadata
|
||||||
|
id: meta
|
||||||
|
uses: docker/metadata-action@v5
|
||||||
|
with:
|
||||||
|
images: ${{ env.IMAGE }}
|
||||||
|
tags: |
|
||||||
|
type=semver,pattern={{version}},value=v${{ env.VERSION }}
|
||||||
|
type=semver,pattern={{major}}.{{minor}},value=v${{ env.VERSION }}
|
||||||
|
type=sha,prefix=,format=short
|
||||||
|
type=raw,value=latest
|
||||||
|
|
||||||
|
- name: Log in to registry
|
||||||
|
uses: docker/login-action@v3
|
||||||
|
with:
|
||||||
|
registry: ${{ vars.REGISTRY_HOST }}
|
||||||
|
username: ${{ gitea.actor }}
|
||||||
|
password: ${{ secrets.TOKEN }}
|
||||||
|
|
||||||
|
- name: Build and push
|
||||||
|
uses: docker/build-push-action@v5
|
||||||
|
with:
|
||||||
|
context: .
|
||||||
|
push: true
|
||||||
|
tags: ${{ steps.meta.outputs.tags }}
|
||||||
|
|
||||||
|
- name: Create Gitea release
|
||||||
|
run: |
|
||||||
|
python3 -c "import json,os; v=os.environ['VERSION']; img=os.environ['IMAGE']; notes=open('/tmp/release_notes.txt').read(); open('/tmp/release_body.json','w').write(json.dumps({'tag_name':'v'+v,'name':'Catalyst v'+v,'body':'### Changes\n\n'+notes+'\n\n### Image\n\n'+img+':'+v,'draft':False,'prerelease':False}))"
|
||||||
|
curl -sf -X POST \
|
||||||
|
-H "Authorization: token ${{ secrets.TOKEN }}" \
|
||||||
|
-H "Content-Type: application/json" \
|
||||||
|
"${{ gitea.server_url }}/api/v1/repos/${{ gitea.repository }}/releases" \
|
||||||
|
--data @/tmp/release_body.json
|
||||||
14
Dockerfile
14
Dockerfile
@@ -1,12 +1,22 @@
|
|||||||
FROM node:lts-alpine
|
FROM node:lts-alpine
|
||||||
|
RUN addgroup -S app && adduser -S app -G app
|
||||||
|
|
||||||
WORKDIR /app
|
WORKDIR /app
|
||||||
|
|
||||||
COPY package*.json ./
|
COPY package*.json ./
|
||||||
RUN npm ci --omit=dev
|
RUN npm ci --omit=dev
|
||||||
|
|
||||||
COPY . .
|
COPY . .
|
||||||
RUN awk -F'"' '/"version"/{printf "const VERSION = \"%s\";\n", $4; exit}' \
|
ARG BUILD_VERSION=""
|
||||||
package.json > js/version.js
|
RUN if [ -n "$BUILD_VERSION" ]; then \
|
||||||
|
printf 'const VERSION = "%s";\n' "$BUILD_VERSION" > js/version.js; \
|
||||||
|
else \
|
||||||
|
awk -F'"' '/"version"/{printf "const VERSION = \"%s\";\n", $4; exit}' \
|
||||||
|
package.json > js/version.js; \
|
||||||
|
fi
|
||||||
|
|
||||||
|
RUN mkdir -p /app/data && chown -R app:app /app
|
||||||
|
USER app
|
||||||
|
|
||||||
EXPOSE 3000
|
EXPOSE 3000
|
||||||
CMD ["node", "server/server.js"]
|
CMD ["node", "server/server.js"]
|
||||||
|
|||||||
@@ -289,6 +289,7 @@ select:focus { border-color: var(--accent); }
|
|||||||
border-radius: 3px;
|
border-radius: 3px;
|
||||||
letter-spacing: 0.08em;
|
letter-spacing: 0.08em;
|
||||||
text-transform: uppercase;
|
text-transform: uppercase;
|
||||||
|
text-align: center;
|
||||||
}
|
}
|
||||||
|
|
||||||
.badge.deployed { background: var(--accent2); color: var(--accent); }
|
.badge.deployed { background: var(--accent2); color: var(--accent); }
|
||||||
|
|||||||
@@ -3,6 +3,7 @@
|
|||||||
<head>
|
<head>
|
||||||
<meta charset="UTF-8">
|
<meta charset="UTF-8">
|
||||||
<meta name="viewport" content="width=device-width, initial-scale=1.0">
|
<meta name="viewport" content="width=device-width, initial-scale=1.0">
|
||||||
|
<base href="/">
|
||||||
<title>Catalyst</title>
|
<title>Catalyst</title>
|
||||||
<link rel="preconnect" href="https://fonts.googleapis.com">
|
<link rel="preconnect" href="https://fonts.googleapis.com">
|
||||||
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
|
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
|
||||||
@@ -176,7 +177,6 @@
|
|||||||
<span id="toast-msg"></span>
|
<span id="toast-msg"></span>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<script src="https://cdnjs.cloudflare.com/ajax/libs/sql.js/1.10.2/sql-wasm.js"></script>
|
|
||||||
<script src="js/version.js" onerror="window.VERSION=null"></script>
|
<script src="js/version.js" onerror="window.VERSION=null"></script>
|
||||||
<script src="js/config.js"></script>
|
<script src="js/config.js"></script>
|
||||||
<script src="js/db.js"></script>
|
<script src="js/db.js"></script>
|
||||||
|
|||||||
@@ -38,6 +38,9 @@ window.addEventListener('popstate', e => {
|
|||||||
|
|
||||||
// ── Bootstrap ─────────────────────────────────────────────────────────────────
|
// ── Bootstrap ─────────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
if (VERSION) document.getElementById('nav-version').textContent = `v${VERSION}`;
|
if (VERSION) {
|
||||||
|
const label = /^\d/.test(VERSION) ? `v${VERSION}` : VERSION;
|
||||||
|
document.getElementById('nav-version').textContent = label;
|
||||||
|
}
|
||||||
|
|
||||||
handleRoute();
|
handleRoute();
|
||||||
|
|||||||
16
package-lock.json
generated
16
package-lock.json
generated
@@ -1,14 +1,15 @@
|
|||||||
{
|
{
|
||||||
"name": "catalyst",
|
"name": "catalyst",
|
||||||
"version": "1.0.3",
|
"version": "1.1.0",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "catalyst",
|
"name": "catalyst",
|
||||||
"version": "1.0.3",
|
"version": "1.1.0",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"express": "^4.18.0"
|
"express": "^4.18.0",
|
||||||
|
"helmet": "^8.1.0"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"jsdom": "^25.0.0",
|
"jsdom": "^25.0.0",
|
||||||
@@ -1958,6 +1959,15 @@
|
|||||||
"node": ">= 0.4"
|
"node": ">= 0.4"
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
"node_modules/helmet": {
|
||||||
|
"version": "8.1.0",
|
||||||
|
"resolved": "https://registry.npmjs.org/helmet/-/helmet-8.1.0.tgz",
|
||||||
|
"integrity": "sha512-jOiHyAZsmnr8LqoPGmCjYAaiuWwjAPLgY8ZX2XrmHawt99/u1y6RgrZMTeoPfpUbV96HOalYgz1qzkRbw54Pmg==",
|
||||||
|
"license": "MIT",
|
||||||
|
"engines": {
|
||||||
|
"node": ">=18.0.0"
|
||||||
|
}
|
||||||
|
},
|
||||||
"node_modules/html-encoding-sniffer": {
|
"node_modules/html-encoding-sniffer": {
|
||||||
"version": "4.0.0",
|
"version": "4.0.0",
|
||||||
"resolved": "https://registry.npmjs.org/html-encoding-sniffer/-/html-encoding-sniffer-4.0.0.tgz",
|
"resolved": "https://registry.npmjs.org/html-encoding-sniffer/-/html-encoding-sniffer-4.0.0.tgz",
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "catalyst",
|
"name": "catalyst",
|
||||||
"version": "1.1.0",
|
"version": "1.2.2",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
"start": "node server/server.js",
|
"start": "node server/server.js",
|
||||||
@@ -9,7 +9,8 @@
|
|||||||
"version:write": "node -e \"const {version}=JSON.parse(require('fs').readFileSync('package.json','utf8'));require('fs').writeFileSync('js/version.js','const VERSION = \\\"'+version+'\\\";\\n');\""
|
"version:write": "node -e \"const {version}=JSON.parse(require('fs').readFileSync('package.json','utf8'));require('fs').writeFileSync('js/version.js','const VERSION = \\\"'+version+'\\\";\\n');\""
|
||||||
},
|
},
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"express": "^4.18.0"
|
"express": "^4.18.0",
|
||||||
|
"helmet": "^8.1.0"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"jsdom": "^25.0.0",
|
"jsdom": "^25.0.0",
|
||||||
|
|||||||
15
server/db.js
15
server/db.js
@@ -133,5 +133,18 @@ export function _resetForTest() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// ── Boot ──────────────────────────────────────────────────────────────────────
|
// ── Boot ──────────────────────────────────────────────────────────────────────
|
||||||
|
// Skipped in test environment — parallel Vitest workers would race to open
|
||||||
|
// the same file, causing "database is locked". _resetForTest() in beforeEach
|
||||||
|
// handles initialisation for every test worker using :memory: instead.
|
||||||
|
|
||||||
init(process.env.DB_PATH ?? DEFAULT_PATH);
|
if (process.env.NODE_ENV !== 'test') {
|
||||||
|
const DB_PATH = process.env.DB_PATH ?? DEFAULT_PATH;
|
||||||
|
try {
|
||||||
|
init(DB_PATH);
|
||||||
|
} catch (e) {
|
||||||
|
console.error('[catalyst] fatal: could not open database at', DB_PATH);
|
||||||
|
console.error('[catalyst] ensure the data directory exists and is writable by the server process.');
|
||||||
|
console.error(e);
|
||||||
|
process.exit(1);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -22,6 +22,9 @@ function validate(body) {
|
|||||||
errors.push(`state must be one of: ${VALID_STATES.join(', ')}`);
|
errors.push(`state must be one of: ${VALID_STATES.join(', ')}`);
|
||||||
if (!VALID_STACKS.includes(body.stack))
|
if (!VALID_STACKS.includes(body.stack))
|
||||||
errors.push(`stack must be one of: ${VALID_STACKS.join(', ')}`);
|
errors.push(`stack must be one of: ${VALID_STACKS.join(', ')}`);
|
||||||
|
const ip = (body.tailscale_ip ?? '').trim();
|
||||||
|
if (ip && !/^(\d{1,3}\.){3}\d{1,3}$/.test(ip))
|
||||||
|
errors.push('tailscale_ip must be a valid IPv4 address or empty');
|
||||||
return errors;
|
return errors;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -75,7 +78,8 @@ router.post('/instances', (req, res) => {
|
|||||||
} catch (e) {
|
} catch (e) {
|
||||||
if (e.message.includes('UNIQUE')) return res.status(409).json({ error: 'vmid already exists' });
|
if (e.message.includes('UNIQUE')) return res.status(409).json({ error: 'vmid already exists' });
|
||||||
if (e.message.includes('CHECK')) return res.status(400).json({ error: 'invalid field value' });
|
if (e.message.includes('CHECK')) return res.status(400).json({ error: 'invalid field value' });
|
||||||
throw e;
|
console.error('POST /api/instances', e);
|
||||||
|
res.status(500).json({ error: 'internal server error' });
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -95,7 +99,8 @@ router.put('/instances/:vmid', (req, res) => {
|
|||||||
} catch (e) {
|
} catch (e) {
|
||||||
if (e.message.includes('UNIQUE')) return res.status(409).json({ error: 'vmid already exists' });
|
if (e.message.includes('UNIQUE')) return res.status(409).json({ error: 'vmid already exists' });
|
||||||
if (e.message.includes('CHECK')) return res.status(400).json({ error: 'invalid field value' });
|
if (e.message.includes('CHECK')) return res.status(400).json({ error: 'invalid field value' });
|
||||||
throw e;
|
console.error('PUT /api/instances/:vmid', e);
|
||||||
|
res.status(500).json({ error: 'internal server error' });
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -109,6 +114,11 @@ router.delete('/instances/:vmid', (req, res) => {
|
|||||||
if (instance.stack !== 'development')
|
if (instance.stack !== 'development')
|
||||||
return res.status(422).json({ error: 'only development instances can be deleted' });
|
return res.status(422).json({ error: 'only development instances can be deleted' });
|
||||||
|
|
||||||
|
try {
|
||||||
deleteInstance(vmid);
|
deleteInstance(vmid);
|
||||||
res.status(204).end();
|
res.status(204).end();
|
||||||
|
} catch (e) {
|
||||||
|
console.error('DELETE /api/instances/:vmid', e);
|
||||||
|
res.status(500).json({ error: 'internal server error' });
|
||||||
|
}
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -1,4 +1,5 @@
|
|||||||
import express from 'express';
|
import express from 'express';
|
||||||
|
import helmet from 'helmet';
|
||||||
import { fileURLToPath } from 'url';
|
import { fileURLToPath } from 'url';
|
||||||
import { dirname, join } from 'path';
|
import { dirname, join } from 'path';
|
||||||
import { router } from './routes.js';
|
import { router } from './routes.js';
|
||||||
@@ -8,6 +9,23 @@ const PORT = process.env.PORT ?? 3000;
|
|||||||
|
|
||||||
export const app = express();
|
export const app = express();
|
||||||
|
|
||||||
|
app.use(helmet({
|
||||||
|
contentSecurityPolicy: {
|
||||||
|
useDefaults: false, // explicit — upgrade-insecure-requests breaks HTTP deployments
|
||||||
|
directives: {
|
||||||
|
'default-src': ["'self'"],
|
||||||
|
'base-uri': ["'self'"],
|
||||||
|
'font-src': ["'self'", 'https://fonts.gstatic.com'],
|
||||||
|
'form-action': ["'self'"],
|
||||||
|
'frame-ancestors': ["'self'"],
|
||||||
|
'img-src': ["'self'", 'data:'],
|
||||||
|
'object-src': ["'none'"],
|
||||||
|
'script-src': ["'self'"],
|
||||||
|
'script-src-attr': ["'unsafe-inline'"], // allow onclick handlers
|
||||||
|
'style-src': ["'self'", 'https://fonts.googleapis.com'],
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}));
|
||||||
app.use(express.json());
|
app.use(express.json());
|
||||||
|
|
||||||
// API
|
// API
|
||||||
|
|||||||
@@ -1,7 +1,8 @@
|
|||||||
import { describe, it, expect, beforeEach } from 'vitest'
|
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest'
|
||||||
import request from 'supertest'
|
import request from 'supertest'
|
||||||
import { app } from '../server/server.js'
|
import { app } from '../server/server.js'
|
||||||
import { _resetForTest } from '../server/db.js'
|
import { _resetForTest } from '../server/db.js'
|
||||||
|
import * as dbModule from '../server/db.js'
|
||||||
|
|
||||||
beforeEach(() => _resetForTest())
|
beforeEach(() => _resetForTest())
|
||||||
|
|
||||||
@@ -237,3 +238,114 @@ describe('DELETE /api/instances/:vmid', () => {
|
|||||||
expect(res.status).toBe(400)
|
expect(res.status).toBe(400)
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
|
// ── Static assets & SPA routing ───────────────────────────────────────────────
|
||||||
|
|
||||||
|
describe('static assets and SPA routing', () => {
|
||||||
|
it('serves index.html at root', async () => {
|
||||||
|
const res = await request(app).get('/')
|
||||||
|
expect(res.status).toBe(200)
|
||||||
|
expect(res.headers['content-type']).toMatch(/html/)
|
||||||
|
})
|
||||||
|
|
||||||
|
it('serves index.html for deep SPA routes (e.g. /instance/117)', async () => {
|
||||||
|
const res = await request(app).get('/instance/117')
|
||||||
|
expect(res.status).toBe(200)
|
||||||
|
expect(res.headers['content-type']).toMatch(/html/)
|
||||||
|
})
|
||||||
|
|
||||||
|
it('serves CSS with correct content-type (not sniffed as HTML)', async () => {
|
||||||
|
const res = await request(app).get('/css/app.css')
|
||||||
|
expect(res.status).toBe(200)
|
||||||
|
expect(res.headers['content-type']).toMatch(/text\/css/)
|
||||||
|
})
|
||||||
|
|
||||||
|
it('does not set upgrade-insecure-requests in CSP (HTTP deployments must work)', async () => {
|
||||||
|
const res = await request(app).get('/')
|
||||||
|
const csp = res.headers['content-security-policy'] ?? ''
|
||||||
|
expect(csp).not.toContain('upgrade-insecure-requests')
|
||||||
|
})
|
||||||
|
|
||||||
|
it('allows inline event handlers in CSP (onclick attributes)', async () => {
|
||||||
|
const res = await request(app).get('/')
|
||||||
|
const csp = res.headers['content-security-policy'] ?? ''
|
||||||
|
// script-src-attr must not be 'none' — that blocks onclick handlers
|
||||||
|
expect(csp).not.toContain("script-src-attr 'none'")
|
||||||
|
})
|
||||||
|
|
||||||
|
it('index.html contains base href / for correct asset resolution on deep routes', async () => {
|
||||||
|
const res = await request(app).get('/')
|
||||||
|
expect(res.text).toContain('<base href="/">')
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
// ── Error handling — unexpected DB failures ───────────────────────────────────
|
||||||
|
|
||||||
|
const dbError = () => Object.assign(
|
||||||
|
new Error('attempt to write a readonly database'),
|
||||||
|
{ code: 'ERR_SQLITE_ERROR', errcode: 8 }
|
||||||
|
)
|
||||||
|
|
||||||
|
describe('error handling — unexpected DB failures', () => {
|
||||||
|
let consoleSpy
|
||||||
|
|
||||||
|
beforeEach(() => {
|
||||||
|
consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
|
||||||
|
})
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
vi.restoreAllMocks()
|
||||||
|
})
|
||||||
|
|
||||||
|
it('POST returns 500 with friendly message when DB throws unexpectedly', async () => {
|
||||||
|
vi.spyOn(dbModule, 'createInstance').mockImplementationOnce(() => { throw dbError() })
|
||||||
|
const res = await request(app).post('/api/instances').send(base)
|
||||||
|
expect(res.status).toBe(500)
|
||||||
|
expect(res.body).toEqual({ error: 'internal server error' })
|
||||||
|
})
|
||||||
|
|
||||||
|
it('POST logs the error with route context when DB throws unexpectedly', async () => {
|
||||||
|
vi.spyOn(dbModule, 'createInstance').mockImplementationOnce(() => { throw dbError() })
|
||||||
|
await request(app).post('/api/instances').send(base)
|
||||||
|
expect(consoleSpy).toHaveBeenCalledWith(
|
||||||
|
expect.stringContaining('POST /api/instances'),
|
||||||
|
expect.any(Error)
|
||||||
|
)
|
||||||
|
})
|
||||||
|
|
||||||
|
it('PUT returns 500 with friendly message when DB throws unexpectedly', async () => {
|
||||||
|
await request(app).post('/api/instances').send(base)
|
||||||
|
vi.spyOn(dbModule, 'updateInstance').mockImplementationOnce(() => { throw dbError() })
|
||||||
|
const res = await request(app).put('/api/instances/100').send(base)
|
||||||
|
expect(res.status).toBe(500)
|
||||||
|
expect(res.body).toEqual({ error: 'internal server error' })
|
||||||
|
})
|
||||||
|
|
||||||
|
it('PUT logs the error with route context when DB throws unexpectedly', async () => {
|
||||||
|
await request(app).post('/api/instances').send(base)
|
||||||
|
vi.spyOn(dbModule, 'updateInstance').mockImplementationOnce(() => { throw dbError() })
|
||||||
|
await request(app).put('/api/instances/100').send(base)
|
||||||
|
expect(consoleSpy).toHaveBeenCalledWith(
|
||||||
|
expect.stringContaining('PUT /api/instances/:vmid'),
|
||||||
|
expect.any(Error)
|
||||||
|
)
|
||||||
|
})
|
||||||
|
|
||||||
|
it('DELETE returns 500 with friendly message when DB throws unexpectedly', async () => {
|
||||||
|
await request(app).post('/api/instances').send({ ...base, stack: 'development', state: 'testing' })
|
||||||
|
vi.spyOn(dbModule, 'deleteInstance').mockImplementationOnce(() => { throw dbError() })
|
||||||
|
const res = await request(app).delete('/api/instances/100')
|
||||||
|
expect(res.status).toBe(500)
|
||||||
|
expect(res.body).toEqual({ error: 'internal server error' })
|
||||||
|
})
|
||||||
|
|
||||||
|
it('DELETE logs the error with route context when DB throws unexpectedly', async () => {
|
||||||
|
await request(app).post('/api/instances').send({ ...base, stack: 'development', state: 'testing' })
|
||||||
|
vi.spyOn(dbModule, 'deleteInstance').mockImplementationOnce(() => { throw dbError() })
|
||||||
|
await request(app).delete('/api/instances/100')
|
||||||
|
expect(consoleSpy).toHaveBeenCalledWith(
|
||||||
|
expect.stringContaining('DELETE /api/instances/:vmid'),
|
||||||
|
expect.any(Error)
|
||||||
|
)
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|||||||
@@ -165,3 +165,23 @@ describe('deleteInstance', () => {
|
|||||||
expect(getInstance(2)).not.toBeNull();
|
expect(getInstance(2)).not.toBeNull();
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// ── Test environment boot isolation ───────────────────────────────────────────
|
||||||
|
|
||||||
|
describe('test environment boot isolation', () => {
|
||||||
|
it('vitest runs with NODE_ENV=test', () => {
|
||||||
|
// Vitest sets NODE_ENV=test automatically. This is the guard condition
|
||||||
|
// that prevents the boot init() from opening the real database file.
|
||||||
|
expect(process.env.NODE_ENV).toBe('test');
|
||||||
|
});
|
||||||
|
|
||||||
|
it('db module loads cleanly in parallel workers without locking the real db file', () => {
|
||||||
|
// Regression: the module-level init(DEFAULT_PATH) used to run unconditionally,
|
||||||
|
// causing "database is locked" when multiple test workers imported db.js at
|
||||||
|
// the same time. process.exit(1) then killed the worker mid-suite.
|
||||||
|
// Fix: boot init is skipped when NODE_ENV=test. _resetForTest() handles setup.
|
||||||
|
// Reaching this line proves the module loaded without calling process.exit.
|
||||||
|
expect(() => _resetForTest()).not.toThrow();
|
||||||
|
expect(getInstances()).toEqual([]);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|||||||
@@ -1,5 +1,7 @@
|
|||||||
// @vitest-environment jsdom
|
// @vitest-environment jsdom
|
||||||
import { describe, it, expect } from 'vitest'
|
import { describe, it, expect } from 'vitest'
|
||||||
|
import { readFileSync } from 'fs'
|
||||||
|
import { join } from 'path'
|
||||||
|
|
||||||
// ── esc() ─────────────────────────────────────────────────────────────────────
|
// ── esc() ─────────────────────────────────────────────────────────────────────
|
||||||
// Mirrors the implementation in ui.js exactly (DOM-based).
|
// Mirrors the implementation in ui.js exactly (DOM-based).
|
||||||
@@ -112,3 +114,53 @@ describe('fmtDateFull', () => {
|
|||||||
expect(fmtDateFull('')).toBe('—')
|
expect(fmtDateFull('')).toBe('—')
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
|
// ── versionLabel() ───────────────────────────────────────────────────────────
|
||||||
|
// Mirrors the logic in app.js — semver strings get a v prefix, dev strings don't.
|
||||||
|
|
||||||
|
function versionLabel(v) {
|
||||||
|
return /^\d/.test(v) ? `v${v}` : v
|
||||||
|
}
|
||||||
|
|
||||||
|
describe('version label formatting', () => {
|
||||||
|
it('prepends v for semver strings', () => {
|
||||||
|
expect(versionLabel('1.1.2')).toBe('v1.1.2')
|
||||||
|
expect(versionLabel('2.0.0')).toBe('v2.0.0')
|
||||||
|
})
|
||||||
|
|
||||||
|
it('does not prepend v for dev build strings', () => {
|
||||||
|
expect(versionLabel('dev-abc1234')).toBe('dev-abc1234')
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
// ── CSS regressions ───────────────────────────────────────────────────────────
|
||||||
|
|
||||||
|
const css = readFileSync(join(__dirname, '../css/app.css'), 'utf8')
|
||||||
|
|
||||||
|
describe('CSS regressions', () => {
|
||||||
|
it('.badge has text-align: center so state labels are not left-skewed on cards', () => {
|
||||||
|
// Regression: badges rendered left-aligned inside the card's flex-end column.
|
||||||
|
// Without text-align: center, short labels (e.g. "deployed") appear
|
||||||
|
// left-justified inside their pill rather than centred.
|
||||||
|
expect(css).toMatch(/\.badge\s*\{[^}]*text-align\s*:\s*center/s)
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
// ── CI workflow regressions ───────────────────────────────────────────────────
|
||||||
|
|
||||||
|
const ciYml = readFileSync(join(__dirname, '../.gitea/workflows/ci.yml'), 'utf8')
|
||||||
|
|
||||||
|
describe('CI workflow regressions', () => {
|
||||||
|
it('build-dev job passes BUILD_VERSION build arg', () => {
|
||||||
|
// Regression: dev image showed semver instead of dev-<sha> because
|
||||||
|
// BUILD_VERSION was never passed to docker build.
|
||||||
|
expect(ciYml).toContain('BUILD_VERSION')
|
||||||
|
})
|
||||||
|
|
||||||
|
it('short SHA is computed with git rev-parse, not $GITEA_SHA (which is empty)', () => {
|
||||||
|
// Regression: ${GITEA_SHA::7} expands to "" on Gitea runners — nav showed "dev-".
|
||||||
|
// git rev-parse --short HEAD works regardless of which env vars the runner sets.
|
||||||
|
expect(ciYml).toContain('git rev-parse --short HEAD')
|
||||||
|
expect(ciYml).not.toContain('GITEA_SHA')
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|||||||
Reference in New Issue
Block a user