Skip to content

Fix/kody language flash on refresh - #1584

Merged
Wellington01 merged 2 commits into
mainfrom
fix/kody-language-flash-on-refresh
Jul 28, 2026
Merged

Fix/kody language flash on refresh#1584
Wellington01 merged 2 commits into
mainfrom
fix/kody-language-flash-on-refresh

Conversation

@Nalu-devs

Copy link
Copy Markdown
Contributor

@railway-app

railway-app Bot commented Jul 22, 2026

Copy link
Copy Markdown

This PR was not deployed automatically as @Nalu-devs does not have access to the Railway project.

In order to get automatic PR deploys, please add @Nalu-devs to your workspace on Railway.

@kody-ai

kody-ai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the `@kody start-review` command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

Preview Environment

Preview environments have been cleaned up.

…ed bridge

Replace per-parameter InitialLanguageConfigProvider with a generic
InitialParametersProvider keyed by ParametersConfigKey. This follows
the existing shell pattern (SSR seed + live query + invalidation):

- SSR-fetch LANGUAGE_CONFIG in settings/layout.tsx and pass via
  initialParameters map to InitialParametersProvider
- In [repositoryId]/layout.tsx, combine liveQuery?.configValue ??
  initialParameter?.configValue ?? ENGLISH (same idiom as shell)
- Add LANGUAGE_CONFIG invalidation in use-code-review-settings-mutation
  so language updates propagate after in-place save
- Revert hook signature change in useOptionalParameterQuery (keep it
  backward-compatible, no options.initialData added)

The generic bridge scales: any future parameter with flash-on-refresh
only needs a new key in the map, not a bespoke provider.
@Nalu-devs
Nalu-devs force-pushed the fix/kody-language-flash-on-refresh branch from 04b6fdf to b105cf1 Compare July 28, 2026 13:21
@kody-ai

kody-ai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the `@kody start-review` command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

- Carry initialTeamId inside InitialParametersBridge so consumers can
  guard the SSR seed against stale-team paints (shell-equivalent pattern
  of effectiveTeamId === initialTeamId).
- Remove two TEMP DEBUG console.log blocks left over from development.
@kody-ai

kody-ai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the `@kody start-review` command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

@Wellington01
Wellington01 merged commit b418dbe into main Jul 28, 2026
13 of 14 checks passed
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.

2 participants