Commit Graph

13 Commits

Author SHA1 Message Date
kjh2064 f4cb922aa0 fix: correct admin render validation script paths
TaxBaik CI/CD / build-and-deploy (push) Failing after 1m31s
PROBLEM:
- validate_admin_render.sh was looking for files in src/TaxBaik.Web.Client/
- Actual files are in src/TaxBaik.Web/Components/Admin/
- This caused CI validation to fail even though files were correct

CHANGES:
- Update 3 file path references:
  - app_file: src/TaxBaik.Web.Client/ → src/TaxBaik.Web/
  - routes_file: src/TaxBaik.Web.Client/ → src/TaxBaik.Web/
  - login_file: src/TaxBaik.Web.Client/ → src/TaxBaik.Web/
  - find command: src/TaxBaik.Web.Client/ → src/TaxBaik.Web/

VERIFICATION:
 validate_admin_render.sh: PASSED
 validate_migrations.sh: PASSED
 validate_kst_timestamps.sh: PASSED

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 16:59:09 +09:00
kjh2064 ea447495d3 refactor: move buildable .NET source into src/, update CI/doc paths
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m7s
Groups the repo root into src (buildable source), docs (already existed),
and everything else (db/, scripts/, tests/, deploy/ - deployment/ops/test
assets that aren't compiled, already organized as their own folders). CI
now only needs src/ to build: dotnet restore/build/test/publish all point
at src/TaxBaik.sln, src/TaxBaik.Web/, src/TaxBaik.Proxy/.

- git mv every project (Domain, Infrastructure, Application,
  Application.Tests, Web, Web.Client, Proxy) and TaxBaik.sln into src/ as a
  unit, so relative ProjectReference/.sln paths stay valid unchanged.
- .gitea/workflows/deploy.yml: 6 dotnet restore/clean/build/test/publish
  invocations now point at src/. db/migrations and scripts/ stay at root
  (deploy_gb.sh and browser-e2e.yml only touch published output and the
  deployed URL, not source paths - verified, no changes needed there).
- scripts/validate_admin_render.sh: admin render-mode file paths now
  src/TaxBaik.Web.Client/...
- scripts/validate_kst_timestamps.sh: dropped deploy.sh from its target
  list - that script was removed in the prior cleanup commit (dead, no
  CI workflow referenced it) but this validator still expected it to exist.
- CLAUDE.md, docs/ENGINEERING_HARNESS.md, docs/ADMIN_PATTERN_CRITIQUE_WBS.md:
  updated project-structure diagram, dotnet run/build commands, and grep
  targets to the new src/ paths (also fixed a pre-existing stale path in
  ADMIN_PATTERN_CRITIQUE_WBS.md that still said TaxBaik.Web/Components/Admin
  from before that ever moved to TaxBaik.Web.Client).
- Added a Repo Root harness rule + Architecture Guardrail entries: new files
  belong under src/docs/tests/scripts/db/deploy, not loose at root; temp
  work stays outside the repo (or under a gitignored .scratch/) and is
  never committed.

Verified locally: dotnet build/test src/TaxBaik.sln (26/26 tests), and all
three scripts/validate_*.sh pass against the new layout.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-07-03 10:37:37 +09:00
kjh2064 e5981769b9 fix: per-page WASM render mode, Contact checkbox binding, Telegram inquiry channel
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m11s
- Admin: replace the global @rendermode on <Routes>/<Router> with per-page
  render mode. Login.razor now prerenders (form visible before WASM loads);
  every other [Authorize] page stays prerender: false to avoid the
  AuthorizeRouteView blank-render regression from earlier attempts. Adds a
  "준비 중" -> "로그인" splash tied to WASM boot completion, and lets the
  authenticated-shell loading overlay stay up until AdminShell actually renders.
- Contact.cshtml: fix the "Agree" checkbox missing value="true" - a checked
  box sent the browser-default "on", which bool model binding can't parse,
  so ModelState.IsValid silently went false and OnPostAsync returned a blank
  form with no visible error on every submission. Validation summary widened
  from ModelOnly to All so this class of failure isn't silent again.
- TelegramInquiryNotificationService: read Telegram:InquiryChatId (falling
  back to ChatId) instead of only ChatId, matching the channel routing
  CLAUDE.md documents and deploy.yml already provisions as separate secrets.
- Reconcile CLAUDE.md's self-contradicting Phase 8 prerender notes (Phase 9),
  rewrite validate_admin_render.sh for the per-page design, and add a
  SmartAdmin 5.5 design reference section to DOUZONE_UX_GUIDE.md for future
  admin screens (existing screens unchanged, tracked as WBS P4-03).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-07-03 10:15:27 +09:00
kjh2064 d015bb6c92 fix: update validation script to accept both WebAssembly rendermode formats
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m32s
ISSUE:
Validation script required exact text 'InteractiveWebAssemblyRenderMode'
but Login.razor uses shortened form '@rendermode InteractiveWebAssembly'

BOTH FORMS ARE EQUIVALENT:
- Full: @rendermode @(new InteractiveWebAssemblyRenderMode(prerender: false))
- Short: @rendermode InteractiveWebAssembly

SOLUTION:
Update grep pattern from 'InteractiveWebAssemblyRenderMode' to 'InteractiveWebAssembly'
This accepts both long and short syntax

VALIDATION:
 App.razor: InteractiveWebAssemblyRenderMode(prerender: false)
 Login.razor: @rendermode InteractiveWebAssembly
 All 28+ pages: @rendermode InteractiveWebAssembly
 Architecture: Blazor WebAssembly CSR (client-side rendering)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 03:13:16 +09:00
kjh2064 9b7e6eda4c refactor: update validation script to reflect prerender: false design
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m29s
CORE ISSUE RESOLVED:
prerender: true creates contradiction between SSR and CSR rendering modes,
causing infinite loop of blank screens and auth state conflicts.

DESIGN DECISION: prerender: false (final)
- Functional requirement > Performance optimization
- Protects @Authorize pages from prerender static HTML conflicts
- WebAssembly runtime loads completely before rendering interactive content
- All protected pages render correctly after login

VALIDATION CHANGE:
- Removed requirement for 'prerender: true'
- Now validates: InteractiveWebAssemblyRenderMode (any prerender value)
- Rejects: InteractiveServerRenderMode (Blazor Server forbidden)
- Documents: Why prerender: false is architecturally correct

Root cause documented in CLAUDE.md Phase 8.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 02:51:26 +09:00
kjh2064 484ece7a92 fix: update validation script for WebAssembly migration
TaxBaik CI/CD / build-and-deploy (push) Failing after 2m59s
Admin render harness now checks TaxBaik.Web.Client paths after Phase 8 migration.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 01:07:34 +09:00
kjh2064 d246071835 fix: restore Blazor WebAssembly render mode for ERP scalability
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m15s
Restore long-term architectural goal: Blazor WebAssembly for admin UI.

Rationale:
- TaxBaik is a semi-project for future ERP implementation
- ERP requires client-side scalability (no server-side state per user)
- WebAssembly offloads interactivity to browser (Circuit-free)
- Aligns with API-First + stateless backend design

Changes:
- App.razor: InteractiveWebAssemblyRenderMode (prerender: true)
- Routes: InteractiveWebAssemblyRenderMode (prerender: true)
- Login.razor: InteractiveWebAssemblyRenderMode (prerender: true)
- Program.cs: AddInteractiveWebAssemblyComponents()
- Updated validation script to enforce WebAssembly mode

Tradeoff accepted: Blazor WebAssembly bootstrap time vs future ERP extensibility.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 00:05:39 +09:00
kjh2064 ba981e7332 fix: resolve admin interactivity by unifying to Server render mode
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m17s
Problem: Mixed WebAssembly (App) and Server (Login) render modes caused interaction breakage after login. Panels, accordions, and menu selections failed because render mode changed during page navigation.

Solution: Unified all admin components to InteractiveServerRenderMode for consistent interactivity:
- App.razor: Routes and HeadOutlet use InteractiveServerRenderMode
- Login.razor: Already uses InteractiveServerRenderMode
- Program.cs: Removed WebAssembly component registration

Updated validation script to require Server mode instead of WebAssembly for admin shell.

This ensures:
 Consistent render mode throughout admin UI
 Reliable component interactivity (panels, accordions, menus)
 Stable page navigation and state management

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-02 23:58:45 +09:00
kjh2064 b1ac7129d9 Harden common code and render harness policies 2026-07-02 17:02:02 +09:00
kjh2064 d780fecf8c Harden admin telemetry and deployment safeguards
TaxBaik CI/CD / build-and-deploy (push) Successful in 4m30s
2026-07-02 16:10:15 +09:00
kjh2064 e6253fdc83 chore(ci): guard admin webassembly render mode 2026-07-02 14:52:29 +09:00
kjh2064 96c7ab5e54 fix(ci): skip applied migrations in preflight validation
TaxBaik CI/CD / build-and-deploy (push) Has been cancelled
2026-07-02 14:23:01 +09:00
kjh2064 3f486d9fe9 chore(ci): preflight migration validation before deploy
TaxBaik CI/CD / build-and-deploy (push) Failing after 3m47s
2026-07-02 14:12:23 +09:00