source-academy / frontend / 8673760081 / 1

31%

master: 31%

push

github

Refactor frontend part 7 (#2846)

* Improve useRef type arguments for HTML elements

* Use named imports for React hooks

* Refactor MobileWorkspace

* Replace if statements with optional invocation
* Move constant `SideContentTab`s out of FC body

* Simplify classnames call

* Remove unnecessary braces around string attributes

Only done for non-UI (non-message/label) strings as these UI strings
will be pulled out for internationalization in the future.

* Deduplicate dependencies

* Create `useTokens` hook

* Update default throw behavior for `useTokens`

* Render SICP chatbot only when logged in

* Fix overloaded types for `useTokens`

Done following default behavior change.

* Use `useTokens` where applicable

* Refactor SICP chatbox

* Use `useTokens`
* Move API call to separate file
* Update React import to default import

* Refactor SICP chatbox further

* Update imports
* Create and use ChatMessage type alias

* Reorganize SICP chatbot files

* Refactor chat completion logic

* Use dependency injection in prompt builder
* Refactor prompt builder logic
* Refactor prompt builder to separate file
* Create type definitions
* Move, type, rename SICP section summaries
* Improve typing

* Refactor SICP chat box component

* Remove unnecessary template literal
* Refactor constants out of component
* Create type definitions
* Create `CONTEXT_SIZE` constant to replace magic numbers
* Refactor logic to use `CONTEXT_SIZE` constant
* Remove unnecessary state variables
* Refactor payload generation
* Move fetching logic from event handler to effect callback instead
* Rename `cleanMessage` to `resetChat`

* Decouple rendering logic from chat completion logic

* Only store string content in `ChatMessage` type
* Move rendering function outside component to prevent unnecessary recreation
* Update render function signature
* Restore GPT-generated output warning for bot messages
* Refactor rend... (continued)

1907 of 6693 branches covered (28.49%)

Branch coverage included in aggregate %.

5249 of 14371 relevant lines covered (36.52%)

21.92 hits per line