Skip to content

Commit 26cc866

Browse files
committed
refactor: drop engine pieces now owned by TechEngine (TechAPI = data + site)
TechAPI is now purely the curated dataset + public site + a lightweight bundled validator. The API server, ingestion crawlers, static-dump generator, coverage checks, full validation and the Pages build/deploy all live in TechEngine (linked as a submodule), so this removes the duplicated pieces to keep the two repos non-overlapping: - remove app/ backend (config, database, main, models, routers, schemas, services), tests/, scripts/ (dump.py, seed.py), docs/ (API design), Dockerfile, docker-compose.yml, .env.example, .dockerignore, pyproject.toml - remove .github/workflows/{deploy-pages,refresh-data,test}.yml (TechEngine owns build/deploy, dump refresh and tests) - keep data/, site/, the lightweight validator (scripts/validate.py -> app/validate.py) and validate-data.yml, which self-checks then chains to TechEngine's full validation Refs #1
1 parent 545e489 commit 26cc866

58 files changed

Lines changed: 51 additions & 5311 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.dockerignore

Lines changed: 0 additions & 18 deletions
This file was deleted.

.env.example

Lines changed: 0 additions & 31 deletions
This file was deleted.

.github/pull_request_template.md

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,16 @@
1-
<!-- PR title must follow Conventional Commits (§14.5), e.g. feat(api): add /compare endpoint -->
1+
<!-- PR title must follow Conventional Commits, e.g. feat(data/cpu): add EPYC Rome SKUs -->
22

33
## What & why
44

55
<!-- What does this change and why? Link issues with "Closes #123". -->
66

7-
## How
7+
## Source
88

9-
<!-- Key implementation notes. -->
10-
11-
## Testing
12-
13-
<!-- How was this verified? Paste test output or screenshots. -->
9+
<!-- Cite the upstream source (vendor product page, Wikipedia infobox, datasheet). -->
1410

1511
## Checklist
1612

17-
- [ ] Type hints on all new functions (§0.5.4)
18-
- [ ] Tests added/updated (§15)
19-
- [ ] `ruff check` and `pytest` pass locally
20-
- [ ] Data changes pass `python -m scripts.validate`
21-
- [ ] SPEC/README updated if behavior changed (§0.5.4)
13+
- [ ] `python -m app.validate` passes locally
14+
- [ ] Files live at the correct `data/<category>/<...>/<slug>.json` path
15+
- [ ] Slugs are kebab-case and unique within the category
16+
- [ ] `source_urls` cites at least one canonical reference

.github/workflows/deploy-pages.yml

Lines changed: 0 additions & 72 deletions
This file was deleted.

.github/workflows/refresh-data.yml

Lines changed: 0 additions & 56 deletions
This file was deleted.

.github/workflows/test.yml

Lines changed: 0 additions & 32 deletions
This file was deleted.
Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,31 @@
11
name: validate-data
22

3+
# Self-check: every PR runs the bundled lightweight validator. Heavy checks
4+
# (coverage gaps, schema migrations, ingestion smoke tests) live in TechEngine.
35
on:
46
pull_request:
57
paths:
68
- "data/**"
9+
- "app/validate.py"
10+
push:
11+
branches: [main]
12+
paths:
13+
- "data/**"
14+
- "app/validate.py"
715

816
jobs:
9-
validate:
17+
self-validate:
1018
runs-on: ubuntu-latest
1119
steps:
1220
- uses: actions/checkout@v4
1321
- uses: actions/setup-python@v5
1422
with:
1523
python-version: "3.12"
16-
- name: Validate seed JSON (§9.3, §15.3)
17-
run: python -m scripts.validate
24+
- name: Self-check (bundled validator)
25+
run: python -m app.validate
26+
27+
engine-validate:
28+
needs: self-validate
29+
uses: GetTechAPI/TechEngine/.github/workflows/validate-data.yml@main
30+
with:
31+
data-ref: ${{ github.event.pull_request.head.sha || github.sha }}

Dockerfile

Lines changed: 0 additions & 28 deletions
This file was deleted.

0 commit comments

Comments
 (0)