LibreKAT/Ocideck
LibreKAT/Ocideck
Watch
4
0
Fork
You've already forked Ocideck
 0
Code Issues 3 Pull requests Projects Releases 4 Packages Activity Actions

[Feature] Link the live web demo from the README, and redeploy it first #589

New issue
Closed
opened 2026-07-22 16:18:09 +00:00 by brenno · 8 comments
brenno commented 2026-07-22 16:18:09 +00:00
Owner
Copy link

Found in the pre-publication first-impression review.

Evidence: https://ocideck.librekat.nl returns 200 and loads to a working welcome screen; /main.dart.js is 11,785,237 bytes. The README never mentions it — the only occurrences of that host in the repo are docs/BUILD.md:124 (as an example query string) and server/fetch-proxy/README.md:52.

Two problems with the deployment itself:

  1. diff web/index.html <served index.html> is 48 lines — the served build predates the form-action none CSP directive now in the repo.
  2. The page stays blank white for roughly ten seconds before the app paints. grep -n "loading|spinner|splash|progress" web/index.html finds nothing; there is no loader.

Why this matters now: a working, zero-install demo of a visual tool already exists and nobody is told. Without it, the cheapest route to "I have seen OciDeck" is installing Flutter 3.44.6 and building — which almost nobody does in the ninety seconds they give a new project, so the entire feature list stays unverified assertion. Publishing while the demo is both unadvertised and running an older CSP than the repo claims is the worst of both.

Proposal: rebuild with make build-web and redeploy; add a splash <div> (logo plus "Loading…") to web/index.html that the Flutter loader removes; then one line under the opening paragraph of the README: "Try it in your browser: … — the same build, running entirely in your tab." Mention that it is the web build and that desktop has more (video, CVE lookup).

Found in the pre-publication first-impression review. **Evidence:** `https://ocideck.librekat.nl` returns 200 and loads to a working welcome screen; `/main.dart.js` is 11,785,237 bytes. The README never mentions it — the only occurrences of that host in the repo are `docs/BUILD.md:124` (as an example query string) and `server/fetch-proxy/README.md:52`. Two problems with the deployment itself: 1. `diff web/index.html <served index.html>` is 48 lines — the served build predates the `form-action none` CSP directive now in the repo. 2. The page stays blank white for roughly ten seconds before the app paints. `grep -n "loading|spinner|splash|progress" web/index.html` finds nothing; there is no loader. **Why this matters now:** a working, zero-install demo of a visual tool already exists and nobody is told. Without it, the cheapest route to "I have seen OciDeck" is installing Flutter 3.44.6 and building — which almost nobody does in the ninety seconds they give a new project, so the entire feature list stays unverified assertion. Publishing while the demo is both unadvertised and running an older CSP than the repo claims is the worst of both. **Proposal:** rebuild with `make build-web` and redeploy; add a splash `<div>` (logo plus "Loading…") to `web/index.html` that the Flutter loader removes; then one line under the opening paragraph of the README: "**Try it in your browser:** … — the same build, running entirely in your tab." Mention that it is the web build and that desktop has more (video, CVE lookup).
brenno commented 2026-07-22 17:30:30 +00:00
Author
Owner
Copy link

Niet meegenomen in #656 (de docs-reparatieronde): dit vraagt iets anders dan tekst — respectievelijk vier schermafdrukken van de draaiende app, en een herbouw plus herdeploy van de webdemo. Blijft open.

Niet meegenomen in #656 (de docs-reparatieronde): dit vraagt iets anders dan tekst — respectievelijk vier schermafdrukken van de draaiende app, en een herbouw plus herdeploy van de webdemo. Blijft open.
brenno commented 2026-07-22 17:36:12 +00:00
Author
Owner
Copy link

Stand na #656 (gemerged), geverifieerd op main: onveranderd open. ocideck.librekat.nl komt nog steeds nul keer voor in de README.

De twee losse punten uit dit issue staan ook nog: de uitgeleverde build loopt achter op de CSP in de repo, en er is geen laadindicator, dus de eerste ~10 seconden zijn wit.

Claim eraf — de tros van #656 is gemerged zonder dit punt.

Stand na `#656` (gemerged), geverifieerd op `main`: **onveranderd open**. `ocideck.librekat.nl` komt nog steeds nul keer voor in de README. De twee losse punten uit dit issue staan ook nog: de uitgeleverde build loopt achter op de CSP in de repo, en er is geen laadindicator, dus de eerste ~10 seconden zijn wit. Claim eraf — de tros van #656 is gemerged zonder dit punt.
brenno commented 2026-07-22 22:39:55 +00:00
Author
Owner
Copy link

De laadscherm-helft is gedaan (PR #704, op main). Het issue blijft open voor de deploy.

De webbouw was de eerste ~10 seconden wit; er ligt nu een laadscherm in web/index.html met logo, "OciDeck" en "Laden… · Loading…". Geen JavaScript, met opzet — het CSP staat geen inline script toe, dus het is een <div> met z-index: -1 die bedekt raakt zodra Flutter verft, en blijft staan als Flutter nooit komt. Een test bewaakt wat make check-web niet ziet: dát het laadscherm er is, en dat het geen inline script draagt (dat zou de CSP schenden en stil niets doen).

Wat er niet in zit, en waarom niet:

  1. de demo herbouwen en herdeployen — de build op ocideck.librekat.nl loopt achter op de CSP in de repo (jouw 48-regel-diff). make build-web maakt de nieuwe bundel; het deployen is jouw hand op de server;
  2. de README-link naar de demo — die hoort pas te landen als de demo herbouwd én de laadindicator live is, anders wijs je bezoekers naar de oude, wit-flitsende build.

Zodra jij hebt gedeployd is de link één regel; zeg het en ik zet hem erbij. Daarvoor blijft dit open.

De laadscherm-helft is gedaan (PR #704, op main). **Het issue blijft open** voor de deploy. De webbouw was de eerste ~10 seconden wit; er ligt nu een laadscherm in `web/index.html` met logo, "OciDeck" en "Laden… · Loading…". Geen JavaScript, met opzet — het CSP staat geen inline script toe, dus het is een `<div>` met `z-index: -1` die bedekt raakt zodra Flutter verft, en blijft staan als Flutter nooit komt. Een test bewaakt wat `make check-web` niet ziet: dát het laadscherm er is, en dat het geen inline script draagt (dat zou de CSP schenden en stil niets doen). **Wat er niet in zit, en waarom niet:** 1. **de demo herbouwen en herdeployen** — de build op `ocideck.librekat.nl` loopt achter op de CSP in de repo (jouw 48-regel-diff). `make build-web` maakt de nieuwe bundel; het deployen is jouw hand op de server; 2. **de README-link naar de demo** — die hoort pas te landen als de demo herbouwd én de laadindicator live is, anders wijs je bezoekers naar de oude, wit-flitsende build. Zodra jij hebt gedeployd is de link één regel; zeg het en ik zet hem erbij. Daarvoor blijft dit open.
brenno commented 2026-07-23 08:39:31 +00:00
Author
Owner
Copy link

De link is https://ocideck.librekat.nl/

De link is https://ocideck.librekat.nl/
brenno commented 2026-07-23 11:34:21 +00:00
Author
Owner
Copy link

/private/tmp/claude-501/-Users-brennodewinter-development-ocideck/ab745412-d678-4f0f-969f-e5f0f97d33e4/scratchpad/589-final.md

/private/tmp/claude-501/-Users-brennodewinter-development-ocideck/ab745412-d678-4f0f-969f-e5f0f97d33e4/scratchpad/589-final.md
brenno commented 2026-07-23 15:00:11 +00:00
Author
Owner
Copy link

Stand vandaag opnieuw geverifieerd: ocideck.librekat.nl levert nog steeds de oude build — het CSP mist form-action 'none' en de referrer-meta, en het laadscherm uit PR #704 zit er niet in (diff met web/index.html bevestigt het). De herbouw is 'make build-web'; het wegzetten op de server (37.97.172.237, niet pawprint) kan alleen jij. De README-link blijft daarop wachten — nu linken wijst bezoekers naar de build zonder laadscherm met tien seconden wit. De reactie hierboven met alleen een scratchpad-pad was een vergissing van een eerdere sessie; het bestand bestaat niet meer.

Stand vandaag opnieuw geverifieerd: ocideck.librekat.nl levert nog steeds de oude build — het CSP mist form-action 'none' en de referrer-meta, en het laadscherm uit PR #704 zit er niet in (diff met web/index.html bevestigt het). De herbouw is 'make build-web'; het wegzetten op de server (37.97.172.237, niet pawprint) kan alleen jij. De README-link blijft daarop wachten — nu linken wijst bezoekers naar de build zonder laadscherm met tien seconden wit. De reactie hierboven met alleen een scratchpad-pad was een vergissing van een eerdere sessie; het bestand bestaat niet meer.
brenno commented 2026-07-23 18:23:19 +00:00
Author
Owner
Copy link

Opgepakt (sessie 7a19ae4a). Stand: README-link staat al op main; laadscherm (PR #704) ook. Rest: verse webbouw ('make build-web') en de deploy naar de server. Ik bouw de bundel en kijk of ik bij de server kan; zo niet, dan ligt de bundel klaar.

Opgepakt (sessie 7a19ae4a). Stand: README-link staat al op main; laadscherm (PR #704) ook. Rest: verse webbouw ('make build-web') en de deploy naar de server. Ik bouw de bundel en kijk of ik bij de server kan; zo niet, dan ligt de bundel klaar.
brenno commented 2026-07-23 18:27:22 +00:00
Author
Owner
Copy link

Afgerond, alle drie de delen geverifieerd:

  1. Laadscherm — PR #704, op main (web/index.html: div zonder inline script, CSP-conform).
  2. Herbouw + herdeploy — vers gebouwd vanaf een schone worktree op main (0c93962e) met 'make build-web', gedeployd naar /var/www/ocideck op de demoserver. Live geverifieerd: index.html is byte-identiek aan de bundel, main.dart.js is de nieuwe (14.608.355 bytes), 'form-action none' en de referrer-meta staan erin, en de app laadt in de browser tot het welkomstscherm zonder consolefouten. De oude build staat als /var/www/ocideck.bak-20260723 op de server (daar zit ook het verdwaalde 96MB .dill-artefact in dat in de webroot lag — dat is nu uit de live map).
  3. README-link — commit 4609ee3c op main, regel 18: 'Try it in your browser' met wat de webbouw niet kan.

Niet meegenomen: het opruimen van de backupmap op de server (bewust, terugdraaipad) en de fetch-proxy — die stond buiten dit issue.

Afgerond, alle drie de delen geverifieerd: 1. **Laadscherm** — PR #704, op main (web/index.html: div zonder inline script, CSP-conform). 2. **Herbouw + herdeploy** — vers gebouwd vanaf een schone worktree op main (0c93962e) met 'make build-web', gedeployd naar /var/www/ocideck op de demoserver. Live geverifieerd: index.html is byte-identiek aan de bundel, main.dart.js is de nieuwe (14.608.355 bytes), 'form-action none' en de referrer-meta staan erin, en de app laadt in de browser tot het welkomstscherm zonder consolefouten. De oude build staat als /var/www/ocideck.bak-20260723 op de server (daar zit ook het verdwaalde 96MB .dill-artefact in dat in de webroot lag — dat is nu uit de live map). 3. **README-link** — commit 4609ee3c op main, regel 18: 'Try it in your browser' met wat de webbouw niet kan. Niet meegenomen: het opruimen van de backupmap op de server (bewust, terugdraaipad) en de fetch-proxy — die stond buiten dit issue.
brenno 2026-07-23 18:27:23 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
codex/markdown-editor-translations
codex/markdown-editor-ux
codex/template-followup
cockpit-wip
v0.1.3
v0.1.2
v0.1.1
v0.1.0
archive/git-mirror
Labels
Clear labels   
accepted
Assessed and will be built

  
bug
Something is broken

  
declined
Assessed and will not be built

  
docs
Documentation only

  
duplicate
Already tracked elsewhere

  
enhancement
New feature or improvement request

  
good first issue
Small, self-contained, and a good place to start

  
in-progress
Someone is working on this right now. Do not start it in parallel.

  
needs-info
Waiting for the reporter

  
privacy
Personal data can leak. Handled before anything else.

  
security
A weakness someone could use. Handled before anything else.

  
triage
Awaiting assessment

No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
LibreKAT/Ocideck#589
No description provided.