Skip to content

fix(site): list software and websites on the homepage - #139

Merged
Seungpyo1007 merged 1 commit into
mainfrom
fix/site-list-new-collections
Jul 31, 2026
Merged

fix(site): list software and websites on the homepage#139
Seungpyo1007 merged 1 commit into
mainfrom
fix/site-list-new-collections

Conversation

@Seungpyo1007

@Seungpyo1007 Seungpyo1007 commented Jul 31, 2026

Copy link
Copy Markdown
Member

What

The homepage endpoint list ended at /v1/brands and never picked up the collections added since — software, and websites (40,084 records, merged this week). The playground's resource picker was missing websites as well, so the two newest collections were invisible on the site even though the API serves them.

  • endpoint list: + /v1/software, + /v1/websites
  • playground picker: + websites
  • meta description: advertised only "smartphone, SoC, GPU and CPU"; now mentions laptops, games, software and websites

Games stay off the list on purpose

deploy-pages strips /v1/games from the published artifact — a million per-record files exceed the Pages deployment window — so listing games would send visitors to a 404. They remain versioned in the repo and served by the engine, just not from the static site.

Verification

Built the site and grepped the output: dist/index.html contains v1/websites and v1/software/index.json, the picker has <option value="websites">, and value="games" appears 0 times — matching what deploy actually publishes.

Refs #1

Closes #19

The endpoint list stopped at brands and never gained the collections added
since: software, and websites (40,084 records, added this week). The playground
resource picker was missing websites too, so the two newest collections were
unreachable from the site even though the API serves them.

Games are deliberately still absent: deploy-pages strips /v1/games from the
published artifact (a million per-record files exceed the Pages deployment
window), so listing them would hand visitors a 404.

Also widens the meta description, which advertised only smartphone/SoC/GPU/CPU.

Verified by building the site and grepping dist/index.html: v1/websites and
v1/software render, games does not.

Refs #1
@Seungpyo1007 Seungpyo1007 added the site Homepage and public site changes label Jul 31, 2026
@Seungpyo1007 Seungpyo1007 self-assigned this Jul 31, 2026
@github-actions github-actions Bot added the bug Something isn't working label Jul 31, 2026
@github-actions github-actions Bot added the enhancement New feature or request label Jul 31, 2026
@Seungpyo1007 Seungpyo1007 moved this from Todo to In Progress in TechAPI-Project Jul 31, 2026
@TechEngineBot

Copy link
Copy Markdown
Member

TechEngine change review: PASS

Check Result
python -m app.validate PASS
python integrity_check.py TechAPI/data --strict PASS
cd TechAPI/site && npm ci && npm run build PASS

Changed data

Category Added Modified Deleted Added verified Added unverified Added Kaggle-sourced
brand 0 0 0 0 0 0
soc 0 0 0 0 0 0
smartphone 0 0 0 0 0 0
tablet 0 0 0 0 0 0
watch 0 0 0 0 0 0
pda 0 0 0 0 0 0
gpu 0 0 0 0 0 0
cpu 0 0 0 0 0 0

Changed record examples

  • No data file changes detected.

Heuristic review

  • Heuristic warnings: none found.

Changed site

Area Added Modified Deleted
homepage/site 0 1 0

Modified site files

  • site/src/pages/index.astro

@TechEngineBot

Copy link
Copy Markdown
Member

TechEngine validation stats: PASS

Data summary

Category Total Verified Unverified Missing verified Tracked Verified % of tracked
brand 207 10 197 0 207 4.8%
soc 2104 123 1981 0 2104 5.8%
smartphone 93396 8453 84943 0 93396 9.1%
tablet 3455 174 3281 0 3455 5.0%
watch 433 11 422 0 433 2.5%
pda 140 27 113 0 140 19.3%
gpu 2030 244 1786 0 2030 12.0%
cpu 3977 986 2991 0 3977 24.8%
all 105742 10028 95714 0 105742 9.5%

Warning

Tracked verified coverage is below 50% for watch 2.5% (11/433), brand 4.8% (10/207), tablet 5.0% (174/3455), soc 5.8% (123/2104), smartphone 9.1% (8453/93396), all 9.5% (10028/105742), gpu 12.0% (244/2030), pda 19.3% (27/140), and 1 more.
Tracked coverage excludes records missing the verified field; see the Missing verified column for those records.
This does not fail validation. Keep imported records verified: false until manual audit, but treat this as follow-up verification work before relying on the affected categories as curated data.

Validation notes

  • Full advisory outlier listings are suppressed on successful runs because they are dataset-wide and mostly stable between PRs.
  • Failure runs still include a detailed log excerpt for debugging.

Key output:

## app.validate
## integrity_check.py --strict
loaded CPU=3977 GPU=2030
✅ integrity gate: no hard anomalies.
Integrity section Flagged lines
structural 0
CPU name/tier consistency (desktop mainstream only) 0
CPU single>multi (cinebench/geekbench — should be multi>=single) 0
CPU era-vs-score outliers 8
CPU cross-source ratio outliers (possible wrong-variant) 152
GPU cross-source ratio outliers + sanity 18

Homepage build:

06:14:44 [build] 2 page(s) built in 836.69s
06:14:44 [build] Complete!

@Seungpyo1007
Seungpyo1007 merged commit 812c1b6 into main Jul 31, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TechAPI-Project Jul 31, 2026
@Seungpyo1007
Seungpyo1007 deleted the fix/site-list-new-collections branch July 31, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request site Homepage and public site changes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Track homepage and site improvements

2 participants