feat: lazy-eval pass_cmd #17

Closed

kbity wants to merge 2 commits from kbity:feature/lazy-eval-pass-cmd into main

kbity commented

2026-04-23 21:23:52 +02:00

Collaborator

closes #8 :)
adds api.platforms.AccountPassword tagged union representing two possible states:

pass_cmd -- password has not yet been evaluated
password -- plaintext password, the command has been evaluated

and integrates them into the existing api.platforms.AccountConfig struct.
evaluation takes place at Wbij.startSessionForAccount.
nb. i had to remove the const qualifier for some *config.Config.Account to allow the union to be mutated.

closes #8 :) adds `api.platforms.AccountPassword` tagged union representing two possible states: - `pass_cmd` -- password has not yet been evaluated - `password` -- plaintext password, the command has been evaluated and integrates them into the existing `api.platforms.AccountConfig` struct. evaluation takes place at `Wbij.startSessionForAccount`. nb. i had to remove the `const` qualifier for some `*config.Config.Account` to allow the union to be mutated.

kbity added 2 commits

2026-04-23 21:23:53 +02:00

[platforms][config] don't immediately evaluate pass_cmd 8d463c8421

[platforms][fzf][config] lazy-eval pass_cmd at Wbij session init 0e2bcb3f40

kbity force-pushed feature/lazy-eval-pass-cmd from 0e2bcb3f40 to c73f30d653

2026-04-24 08:03:32 +02:00

Compare

maleszka commented

2026-04-24 10:26:29 +02:00

Owner

Thank you! Good work!

Pass cmd should be evaluated at the front-end side, not api side. api module is intended to be extracted out into a separate repository and be an universal library for interacting with competitive programming platforms. Therefore, platforms.AccountConfig should be a "compiled" object that does not make any assumptions about the environment.

I will create a follow-up PR with small refactor.

Thank you! Good work! Pass cmd should be evaluated at the front-end side, not api side. `api` module is intended to be extracted out into a separate repository and be an universal library for interacting with competitive programming platforms. Therefore, `platforms.AccountConfig` should be a "compiled" object that does not make any assumptions about the environment. I will create a follow-up PR with small refactor.

👍 1