kleb/kBackup
Watch
1
0
Fork
You've already forked kBackup
 0
Code Issues Pull requests Projects Releases 4 Packages Wiki Activity Actions

Add TUI-managed web setup #4

Closed
kleb wants to merge 1 commit from feature/tui-web-setup into main
pull from: feature/tui-web-setup
merge into: kleb:main
Conversation 1 Commits 1 Files changed 36
kleb commented 2026-07-27 13:20:50 +02:00
Owner
Copy link

Summary

  • add one-time TUI onboarding for a loopback-only web interface
  • support live transactional web enable/disable and persisted advanced settings
  • pass first-setup owner tokens in URL fragments with immediate browser removal
  • expose all safe service settings in the TUI and web settings page
  • update documentation, generated protobuf clients, and embedded web assets

Validation

  • go vet ./...
  • go test -count=1 ./...
  • go test -count=1 -race ./internal/cache ./internal/repo ./internal/job ./internal/notify ./internal/scheduler ./internal/web
  • targeted race tests for dynamic web settings
  • npm --prefix web run check
  • generated protobuf verification
  • embedded web distribution verification
  • git diff --check

Browser automation was unavailable in the local environment; fragment and setup-page behavior is covered by Vitest component tests.

## Summary - add one-time TUI onboarding for a loopback-only web interface - support live transactional web enable/disable and persisted advanced settings - pass first-setup owner tokens in URL fragments with immediate browser removal - expose all safe service settings in the TUI and web settings page - update documentation, generated protobuf clients, and embedded web assets ## Validation - `go vet ./...` - `go test -count=1 ./...` - `go test -count=1 -race ./internal/cache ./internal/repo ./internal/job ./internal/notify ./internal/scheduler ./internal/web` - targeted race tests for dynamic web settings - `npm --prefix web run check` - generated protobuf verification - embedded web distribution verification - `git diff --check` Browser automation was unavailable in the local environment; fragment and setup-page behavior is covered by Vitest component tests.
kAgent commented 2026-07-27 13:20:51 +02:00
Collaborator
Copy link

kReview review

Verdict: no findings

No findings to address in the reviewed diff.

No concrete correctness, security, or lifecycle regression is provable from the visible diff and supplied repository context.

Excluded as generated or vendored (not reviewed): web/dist/assets/index-3ks0bc2G.js, web/dist/assets/index-B6JIooyo.css, web/dist/assets/index-O-Iazc_C.js, web/dist/assets/index-h0dXiOvq.css, web/dist/index.html.

Reviewed by kReview at fd91a6bbaf. This comment is conservative and based only on the PR diff, metadata, and supplied repository context.

Est. cost ~$1.38 (598.0k in / 3.4k out / gpt-5.6-sol).

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:fd91a6bbafb07397b4f872bb5cd53265ce7ea57e --> ## kReview review **Verdict:** no findings No findings to address in the reviewed diff. No concrete correctness, security, or lifecycle regression is provable from the visible diff and supplied repository context. _Excluded as generated or vendored (not reviewed):_ `web/dist/assets/index-3ks0bc2G.js`, `web/dist/assets/index-B6JIooyo.css`, `web/dist/assets/index-O-Iazc_C.js`, `web/dist/assets/index-h0dXiOvq.css`, `web/dist/index.html`. _Reviewed by kReview at `fd91a6bbaf`. This comment is conservative and based only on the PR diff, metadata, and supplied repository context._ _Est. cost ~$1.38 (598.0k in / 3.4k out / gpt-5.6-sol)._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiZmQ5MWE2YmJhZmIwNzM5N2I0Zjg3MmJiNWNkNTMyNjVjZTdlYTU3ZSIsInN1bW1hcnkiOiJObyBjb25jcmV0ZSBjb3JyZWN0bmVzcywgc2VjdXJpdHksIG9yIGxpZmVjeWNsZSByZWdyZXNzaW9uIGlzIHByb3ZhYmxlIGZyb20gdGhlIHZpc2libGUgZGlmZiBhbmQgc3VwcGxpZWQgcmVwb3NpdG9yeSBjb250ZXh0LiIsImZpbmRpbmdzIjpbXSwiY3VtdWxhdGl2ZUNvc3QiOnsidXNkIjoxLjM4Mzg1Mjk5OTk5OTk5OTgsImlucHV0VG9rZW5zIjo1OTgwNDYsIm91dHB1dFRva2VucyI6MzQyOSwibW9kZWwiOiJncHQtNS42LXNvbCJ9fQ== -->
kleb closed this pull request 2026-07-27 13:26:11 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
kleb/kBackup!4
No description provided.