Skip to content

chore: refresh uv.lock to match pyproject version - #29

Merged
Bre77 merged 1 commit into
mainfrom
fm/pts-uv-lock-refresh
Aug 12, 2026
Merged

chore: refresh uv.lock to match pyproject version#29
Bre77 merged 1 commit into
mainfrom
fm/pts-uv-lock-refresh

Conversation

@Bre77

Copy link
Copy Markdown
Member

uv.lock's self-referential teslemetry-stream entry was still pinned at 0.10.0 after pyproject.toml moved to 0.10.1, so uv sync reported a stale lockfile. Refreshes it to match.

@Bre77Bre77 added the fm Opened by a Firstmate crewmate label Aug 12, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

self.stream.async_add_listener(
self._on_config_event,
{Key.VIN: self.vin, Key.CONFIG: None},
internal=True,
)

P2 Badge Capture config events before constructing vehicles

When a generic public listener discovers an uncached VIN and calls get_vehicle() while handling that VIN's config event, listen() has already snapshotted the listener registry, so this newly registered _on_config_event is not invoked for the current event. If no later config change arrives, the vehicle remains at fields={} and preferTyped=None even though the stream received the authoritative configuration, defeating the promised fresh config record; seed the new vehicle from the current event or register config capture at the stream level before dispatch.

AGENTS.md reference: AGENTS.md:L19-L19

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Bre77
Bre77 merged commit 679f845 into mainAug 12, 2026
14 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fmOpened by a Firstmate crewmate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@Bre77