Skip to content

feat: improve beginner onboarding and model config sync - #1445

Open
zavier-bin wants to merge 1 commit into
NoFxAiOS:devfrom
zavier-bin:codex/week1-full-sync
Open

feat: improve beginner onboarding and model config sync#1445
zavier-bin wants to merge 1 commit into
NoFxAiOS:devfrom
zavier-bin:codex/week1-full-sync

Conversation

@zavier-bin

Copy link
Copy Markdown
Contributor

Summary

This PR uploads the full current nofx-dev-week1 frontend onboarding and model configuration updates on top of dev.

The main goal is to make the beginner model setup flow consistent and remove refresh-dependent UI behavior after model configuration changes.

What Changed

Beginner onboarding flow

  • beginner mode now shows the welcome onboarding route consistently instead of depending on a separate persisted completion flag
  • the old local beginner_onboarding_completed flag helpers were removed
  • continuing from the beginner onboarding page now relies on navigation flow instead of writing that extra local completion flag

Claw402 first-time setup

  • saving the Claw402 model modal now auto-runs the existing beginner onboarding preparation flow when no wallet exists yet
  • first-time users no longer need a separate one-click generation step after pressing save
  • the returned beginner wallet address is persisted and reused in the trader page state
  • the default beginner model is set to deepseek

Model configuration sync without refresh

  • added a lightweight model-configs-updated event helper
  • model updates and beginner onboarding preparation now emit that event
  • the following screens now reactively refresh model-dependent UI without a manual page reload:
    • AI traders page and AI model status panel
    • strategy studio model list
    • Telegram config modal

Beginner guide copy

  • updated the beginner guide so it points users to open the model modal and save directly
  • removed the misleading separate one-click setup wording now that save already completes first-time setup

Claw402 modal UX

  • first-time Claw402 save is allowed even without a manually entered private key
  • helper copy explains that the wallet will be auto-generated on first save
  • validation and submit button state now support both existing-wallet reuse and first-time auto-generation

Strategy activation error handling

  • strategy activation now surfaces backend error messages instead of always showing a generic failure message

Testing

  • docker compose build nofx-frontend

Notes

  • This PR is intended to represent the current full state of the nofx-dev-week1 workspace for these onboarding and model-sync related changes.

@cla-assistant

cla-assistant Bot commented Mar 30, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


apple seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions

Copy link
Copy Markdown

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟡 Medium (355 lines: +194 -161)

🔧 Backend Checks

Go Formatting: ⚠️ Needs formatting

Files needing formatting
api/handler_wallet.go
api/utils_test.go
config/config.go
kernel/engine.go
kernel/formatter.go
kernel/grid_engine.go
kernel/validate_test.go
main.go
manager/trader_manager.go
market/data.go

Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

📖 Resources

Questions? Feel free to ask in the comments! 🙏


These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant