docs(round2): emails, SDK CDN delivery, wire orphan diagrams#17
Merged
Conversation
1. Contact email: replace info@app.fivucsas.com and security@fivucsas.com with info@fivucsas.com (primary) + rollingcat.help@gmail.com (alternate) in SECURITY.md and 02-architecture/security.md. 2. SDK delivery: npm packages @fivucsas/auth-* are not yet published. Update Quick Start, Option A/B/C and tenant-onboarding to use the confirmed CDN URLs (fivucsas-auth.js + fivucsas-auth.esm.js — both HTTP 200 at https://verify.fivucsas.com/). Old /sdk/ sub-path was 404. Mark npm install lines as "coming soon — not yet published". 3. Orphan diagrams: wired all 11 .drawio and 5 ADD_diagrams/*.png into new §11–12 of ARCHITECTURE_DIAGRAMS.md and README.md summary table. Per-diagram verdict: accurate+wired (7 drawio, 3 PNG) vs needs-regen (3 drawio, 2 PNG — NGINX→Traefik, stale progress bars). Corrected HS512→RS256 in system_architecture.drawio and Mermaid seq diagram in ARCHITECTURE_DIAGRAMS.md. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…design-doc refs) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Emails: Replaced
info@app.fivucsas.comandsecurity@fivucsas.comeverywhere withinfo@fivucsas.com(primary) +rollingcat.help@gmail.com(alternate). Files:SECURITY.md,02-architecture/security.md.SDK delivery: npm packages
@fivucsas/auth-*return 404 (not published). The CDN is live:https://verify.fivucsas.com/fivucsas-auth.js(IIFE, HTTP 200) andhttps://verify.fivucsas.com/fivucsas-auth.esm.js(ESM, HTTP 200). UpdatedINTEGRATION_GUIDE.md(Quick Start + Options A/B/C),tenant-onboarding.md, andEMBEDDABLE_AUTH_WIDGET_ARCHITECTURE.mdto use the confirmed CDN paths and mark npm as "coming soon — not yet published".Orphan diagrams wired: All 11
.drawioand 5ADD_diagrams/*.pngreferenced in new §11–12 ofARCHITECTURE_DIAGRAMS.mdand a summary table in02-architecture/README.md. Per-diagram verdicts below.Diagram verdicts
.drawio— accurate + wiredsystem_architecture.drawio— JWT corrected HS512 → RS256 in source XMLhexagonal_architecture.drawio— accurateer_diagram.drawio— broadly accurate (pre-V55 schema; note on V55–V60 added)use_case_diagram.drawio— accuratebiometric_puzzle_flow.drawio— accuratedata_flow_diagram.drawio— accuratenfc_handshake.drawio— accurateml_pipeline.drawio— accurate (9 DeepFace models shown; MobileFaceNet was a separate stripped model not listed here).drawio— needs regeneration (wired with caveat note)deployment_architecture.drawio— shows NGINX; prod uses Traefiktechnology_stack.drawio— API gateway section shows NGINX; prod uses Traefikimplementation_progress.drawio— "Fall 2025" percentages; project is now 98%+ completeADD_diagrams/*.png— accurate + wireddiagram_01_use_cases_by_actor.pngdiagram_02_domain_model___core_entities.pngdiagram_03_entity_relationship_diagram.pngADD_diagrams/*.png— needs regenerationdiagram_04_system_architecture_overview.png— NGINX gateway + "DeepFace 9 Models" (MobileFaceNet removed); Traefik is the real gatewaydiagram_05_docker_deployment.png— nginx:8000 + web-app:5173; prod uses Traefik + verify.fivucsas.comTest plan
curl -sI https://verify.fivucsas.com/fivucsas-auth.js→ 200;curl -sI https://verify.fivucsas.com/fivucsas-auth.esm.js→ 200info@app.fivucsas.comorsecurity@fivucsas.comin docs treediagrams/system_architecture.drawioin draw.io and confirm JWT label reads RS256ARCHITECTURE_DIAGRAMS.md— all relative paths should resolve within the repo🤖 Generated with Claude Code