Skip to content

chore(tauri): refresh shell lockfile#2398

Draft
Bortlesboat wants to merge 1 commit into
tinyhumansai:mainfrom
Bortlesboat:codex/OSS-1-tauri-lockfile-deps
Draft

chore(tauri): refresh shell lockfile#2398
Bortlesboat wants to merge 1 commit into
tinyhumansai:mainfrom
Bortlesboat:codex/OSS-1-tauri-lockfile-deps

Conversation

@Bortlesboat
Copy link
Copy Markdown
Contributor

Summary

  • Refreshes �pp/src-tauri/Cargo.lock for the Tauri shell dependency graph.
  • Adds the transitive lock entries Cargo currently requires for the checked-in �pp/src-tauri/Cargo.toml graph.
  • Keeps the change lockfile-only; no source or manifest behavior changes.

Problem

  • From a clean checkout, cargo check --manifest-path app/src-tauri/Cargo.toml --locked fails because Cargo needs to update �pp/src-tauri/Cargo.lock.
  • That makes locked local/CI-style validation impossible until the lockfile is refreshed.

Solution

  • Ran the Tauri shell cargo check without --locked to let Cargo add the missing transitive packages.
  • Re-ran the same Tauri shell check with --locked to verify the refreshed lockfile is complete.

Submission Checklist

  • Tests added or updated: N/A - lockfile-only dependency graph refresh; regression is covered by the previously failing locked cargo check.
  • Diff coverage >= 80%: N/A - lockfile-only change with no executable changed lines.
  • Coverage matrix updated: N/A - no feature row added, removed, or renamed.
  • All affected feature IDs from the matrix are listed in the PR description under ## Related: N/A - no feature matrix IDs affected.
  • No new external network dependencies introduced: no manifest dependency was added; this only records transitive dependencies already required by the checked-in Tauri graph.
  • Manual smoke checklist updated: N/A - does not change release smoke behavior.
  • Linked issue closed via Closes #NNN in the ## Related section: N/A - no existing issue found for this lockfile drift.

Impact

  • Developer/CI impact only: locked Tauri shell cargo checks can run from a clean checkout.
  • No runtime, migration, security, or user-visible behavior change intended.

Related

  • Closes: N/A
  • Follow-up PR(s)/TODOs: N/A

AI Authored PR Metadata (required for Codex/Linear PRs)

Linear Issue

  • Key: N/A - opportunistic setup finding while installing and verifying OpenHuman locally.
  • URL: N/A

Commit & Branch

  • Branch: codex/OSS-1-tauri-lockfile-deps
  • Commit SHA: $sha

Validation Run

  • [x]
    ode scripts/codex-pr-preflight.mjs --lightweight - passed.
  • pnpm typecheck - passed.
  • Focused tests: cargo check --manifest-path app/src-tauri/Cargo.toml --locked - failed before this lockfile refresh, passed after it.
  • Rust fmt/check (if changed): cargo check --manifest-path Cargo.toml passed; cargo fmt --manifest-path app/src-tauri/Cargo.toml --all --check is blocked by existing vendored newline-style baseline, listed below.
  • Tauri fmt/check (if changed): cargo check --manifest-path app/src-tauri/Cargo.toml --locked passed; full format check is blocked as listed below.
  • git diff --check - passed.

Validation Blocked

  • command: node scripts/codex-pr-preflight.mjs --strict-path --lightweight
  • error: expected repo path /workspace/openhuman, got C:\Users\andre\OneDrive\Documents\Github\openhuman
  • impact: Re-ran without --strict-path because this is a native Windows checkout, not Codex web's /workspace path.
  • command: cargo fmt --manifest-path app/src-tauri/Cargo.toml --all --check
  • error: baseline vendored submodule files under app/src-tauri/vendor/tauri-cef report Incorrect newline style
  • impact: Did not normalize vendored files in this lockfile-only PR.
  • command: pnpm --filter openhuman-app format:check
  • error: baseline Prettier check reports code style issues in 910 app files before reaching this lockfile-only change
  • impact: Left baseline formatting untouched; changed file is Cargo.lock only.
  • command: git push
  • error: pre-push hook auto-ran repo-wide formatting after the unrelated baseline format failure and dirtied the working tree
  • impact: Restored the unrelated churn, then pushed with --no-verify per repo guidance for unrelated pre-existing hook breakage.

Behavior Changes

  • Intended behavior change: No app behavior change; lockfile now matches the Tauri shell dependency graph.
  • User-visible effect: None.

Parity Contract

  • Legacy behavior preserved: Yes - source manifests and runtime code are unchanged.
  • Guard/fallback/dispatch parity checks: N/A - no dispatch, guard, or fallback logic changed.

Duplicate / Superseded PR Handling

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 106662b9-ec33-46d1-ab9c-4bedaf78cf43

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant