Master update - #3725
Open
GuillaumeDSM wants to merge 87 commits into
Open
Master update#3725GuillaumeDSM wants to merge 87 commits into
GuillaumeDSM wants to merge 87 commits into
Conversation
Interactive Cloud Agent shells resolve `node` to the platform-provided Node (v22 via /exec-daemon/node) which precedes /usr/bin/node in PATH, even though the repo pins Node 24 (.nvmrc, .cursor/Dockerfile). Make the generated .cursor/env.sh prepend the directory of a system node whose major matches .nvmrc, ahead of the platform node, while keeping the venv bin first so python/OctoBot still resolve to the OctoBot venv. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Optional max_retry_interval samples wait between condition checks using the same randrange semantics as wait(). Re-sample on each scheduling cycle in return_remaining_time and blocking modes. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Assert max_retry_interval in generated DSL when LOOP_INTERVAL_MAX is set and preserve baseline behavior without it. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
ActionsDAGParser emits max_retry_interval when LOOP_INTERVAL_MAX exceeds LOOP_INTERVAL and validates max is not below min. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Structured auth error codes on the node API (401/503 detail.code) and plain-language login UI: inline error, contextual tips, session-expired banner, verify-before-persist session. UTF-8 Basic auth helper; recovery screen copy clarifying broken local sign-in vs wrong passphrase. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Bump @drakkar.software/octobot-protocol consumer range to ^0.9.0 and refresh the workspace package-lock.json so npm ci stays in sync. Co-authored-by: Herklos <herklos@drakkar.software>
Replace stacked quiz inputs with the same numbered grid as reveal. Only pickSeedQuizPositions cells are editable; blur validation shows field errors or morphs correct answers into confirmed chips. Continue stays disabled until all three quiz words are confirmed. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Read event.target.value on blur so validation runs after the latest keystroke instead of stale answers state from the previous render. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Use a fixed h-11 cell shell for plaintext, quiz inputs, and confirmed chips. Style quiz inputs borderless inside the cell and reserve a footer row for field errors so grid rows stay aligned. Widen Add wallet seed onboarding and setup seed flow containers for the 12-word grid. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Add shared cryptoSecretInputProps and apply to seed quiz cells, Settings import seed/private key fields, and setup private key import. Passphrase fields unchanged. Vitest covers the helper and quiz markup. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Implement passphrase reset via BIP39 seed or hex private key proof, unauthenticated recover API endpoint, dual-bucket rate limiting, and /login/recover-seed UI with success banner on return to login. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Split /login into a layout route with Outlet so /login/recover-seed renders the recovery form instead of the unlock screen. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Do not upgrade the Nth failed recovery response to 429; the entry is_rate_limited check blocks follow-up attempts. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Move recover back link below CTA, drop prove-ownership subheader, and use proof-field placeholders. Show post-recover success via Sonner toast on login instead of an inline banner. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Move dual-bucket failure limiting into node_api_interface core helper; wire recover-from-seed through a thin recovery policy module. RAM-only counters remain per-process (no shared store). Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Move InProcessFailureRateLimiter to octobot_commons. HTTPRateLimiter subclasses it and raises HTTP 429; recover-from-seed uses http_failure_rate_limited with dual-bucket policies unchanged. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com> docs(commons): docstrings for in-process rate limiter methods Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com> refactor(node-api): stack recover rate-limit decorator on route Use @http_failure_rate_limited on recover_wallet_from_seed_route with router.post outermost; HTTP wallet mapping via a thin wrapper decorator. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com> fix(node-ui): remove leftover merge conflict markers in login.tsx Keep Outlet-only login layout; login and recover-seed remain child routes. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Herklos
approved these changes
Oct 1, 2026
This branch has not been deployed
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
3 participants
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.
No description provided.