fix(ci): migrate .trunk/trunk.yaml to the 0.1 schema

Trunk Check has been failing on every PR with config-error findings:
the legacy pre-0.1 schema keys (linters/formatters/actions/cache/env)
are unrecognized by the pinned CLI 1.22.2. Migrate to the modern
schema, preserving the repo's linting intent:

- version: 0.1, cli.version: 1.22.2, plugin source uri added
- lint.definitions: repo-specific command overrides (black line length
  100 for lint+format, clippy --all-targets --all-features -D warnings)
  merged over the trunk plugin definitions
- lint.enabled: same 12 tools as before (plugin defaults otherwise)
- lint.ignore: scope prettier/yamllint off .github/workflows/** - those
  files are covered by actionlint; prettier re-indents `on:`-keyed
  workflow files with a nonstandard style and yamllint's 80-column
  default flags long action refs
- actions.enabled: trunk-check-pre-push (modern successor of the old
  trunk-check action)

Also fix the issues the freshly-running tools surfaced on this branch's
diff:
- actionlint: drop invalid `deployment` key under `environment` and
  quote the `--build_metadata=COMMIT_SHA=$(git rev-parse HEAD)` array
  element (SC2207) in v8-canary.yml and rusty-v8-release.yml
- taplo: add .taplo.toml declaring the repo's 4-space array indent and
  expand the tonic features array to match taplo's output
- add .trunk/.gitignore for trunk's local runtime state (canonical set
  from trunk-io/plugins)

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>

Author: KooshaPari

.github/workflows/rusty-v8-release.yml

@@ -66,7 +66,6 @@ jobs:
       actions: read
     environment:
       name: bazel
-      deployment: false
     strategy:
       fail-fast: false
       matrix:
@@ -202,7 +201,7 @@ jobs:
             --config=rusty-v8-upstream-libcxx
             "--config=v8-target-${V8_CPU}"
             "${pair_target}"
-            --build_metadata=COMMIT_SHA=$(git rev-parse HEAD)
+            "--build_metadata=COMMIT_SHA=$(git rev-parse HEAD)"
           )
           if [[ "${SANDBOX}" != "true" ]]; then
             bazel_args+=(--config=v8-release-compat)

.github/workflows/v8-canary.yml

@@ -110,7 +110,6 @@ jobs:
       actions: read
     environment:
       name: bazel
-      deployment: false
     strategy:
       fail-fast: false
       matrix:
@@ -245,7 +244,7 @@ jobs:
             "--config=v8-target-${V8_CPU}"
             "${pair_target}"
             --verbose_failures
-            --build_metadata=COMMIT_SHA=$(git rev-parse HEAD)
+            "--build_metadata=COMMIT_SHA=$(git rev-parse HEAD)"
           )
           if [[ "${SANDBOX}" != "true" ]]; then
             bazel_args+=(--config=v8-release-compat)

.taplo.toml

@@ -0,0 +1,6 @@
+# TOML formatting style for this repository (used by the `taplo` linter in
+# .trunk/trunk.yaml). The existing Cargo.toml files indent array entries
+# with 4 spaces, so keep taplo aligned with that instead of its 2-space
+# default.
+[formatting]
+indent_string = "    "

.trunk/.gitignore

@@ -0,0 +1,8 @@
+*out
+*logs
+*actions
+*notifications
+*tools
+plugins
+user_trunk.yaml
+user.yaml

.trunk/trunk.yaml

@@ -1,3 +1,4 @@
+---
 # =============================================================================
 # Trunk.io — Plugin versions for linting/formatting tools
 # =============================================================================
@@ -9,60 +10,72 @@ version: 0.1
 
 # CLI version pinned for reproducible runs (matches the version CI downloads).
 cli:
-  version: 1.22.2
+    version: 1.22.2
 
 # The `trunk` plugin provides the linter/formatter definitions used below.
 # User definitions merge over the plugin: fields set in this file override the
 # plugin, everything else (files, tools, runtimes, version pins) is inherited.
 plugins:
-  sources:
-    - id: trunk
-      ref: v1.2.2
-      uri: https://github.com/trunk-io/plugins
+    sources:
+        - id: trunk
+          ref: v1.2.2
+          uri: https://github.com/trunk-io/plugins
 
 lint:
-  # Repo-specific command overrides merged over the plugin definitions.
-  definitions:
-    # black: enforce 100-char line length in both lint and format modes.
-    - name: black
-      commands:
-        - name: lint
-          output: pass_fail
-          run: black --check --line-length 100 ${target}
-          success_codes: [0, 1]
-        - name: format
-          output: rewrite
-          run: black --line-length 100 ${target}
-          success_codes: [0]
-          batch: true
-          in_place: true
-          formatter: true
-    # clippy: lint all targets/features and fail on any warning.
-    - name: clippy
-      commands:
-        - name: lint
-          output: clippy
-          target: ${parent_with(Cargo.toml)}
-          run: cargo clippy --message-format json --all-targets --all-features --locked -- -D warnings
-          success_codes: [0, 101, 383]
-          disable_upstream: true
+    # Repo-specific command overrides merged over the plugin definitions.
+    definitions:
+        # black: enforce 100-char line length in both lint and format modes.
+        - name: black
+          commands:
+              - name: lint
+                output: pass_fail
+                run: black --check --line-length 100 ${target}
+                success_codes: [0, 1]
+              - name: format
+                output: rewrite
+                run: black --line-length 100 ${target}
+                success_codes: [0]
+                batch: true
+                in_place: true
+                formatter: true
+        # clippy: lint all targets/features and fail on any warning.
+        - name: clippy
+          commands:
+              - name: lint
+                output: clippy
+                target: ${parent_with(Cargo.toml)}
+                run: >
+                    cargo clippy --message-format json --all-targets
+                    --all-features
+                    --locked -- -D warnings
+                success_codes: [0, 101, 383]
+                disable_upstream: true
 
-  # Enabled tools (plugin defaults unless overridden above).
-  enabled:
-    - actionlint
-    - black
-    - clippy
-    - eslint
-    - golangci-lint
-    - mypy
-    - prettier
-    - ruff
-    - rustfmt
-    - shellcheck
-    - taplo
-    - yamllint
+    # Enabled tools (plugin defaults unless overridden above).
+    enabled:
+        - actionlint
+        - black
+        - clippy
+        - eslint
+        - golangci-lint
+        - mypy
+        - prettier
+        - ruff
+        - rustfmt
+        - shellcheck
+        - taplo
+        - yamllint
+
+    # GitHub workflow files are covered by actionlint (syntax + shellcheck).
+    # prettier's YAML printer re-indents `on:`-keyed workflow files with a
+    # nonstandard style, and yamllint's 80-column default flags long action
+    # refs; neither reflects this repo's intent, so scope both off workflows.
+    ignore:
+        - linters: [prettier, yamllint]
+          paths:
+              - .github/workflows/**
 
 # Git hooks: run `trunk check` before `git push`.
 actions:
-  enabled:
-    - trunk-check-pre-push
+    enabled:
+        - trunk-check-pre-push

codex-rs/Cargo.toml

@@ -373,7 +373,10 @@ tokio-tungstenite = { version = "0.28.0", features = [
     "rustls-tls-native-roots",
 ] }
 tokio-util = "0.7.18"
-tonic = { version = "0.14.3", default-features = false, features = ["channel", "codegen"] }
+tonic = { version = "0.14.3", default-features = false, features = [
+    "channel",
+    "codegen",
+] }
 tonic-prost = "0.14.3"
 toml = "0.9.5"
 toml_edit = "0.24.0"