[codex] Align implicit skill reads with parser

[alexsong-oai]

Summary

• reuse the shared shell read parser for implicit skill doc invocation detection
• add regression coverage for nl -ba .../SKILL.md

Why

Desktop could render Read User Context skill for reads recognized by the shared command parser, while implicit skill_invocation analytics used a separate reader allowlist and missed cases such as nl.

Validation

• HOME=/private/tmp/codex-core-skills-home-pr PATH=/Users/alexsong/.cache/cargo-home/bin:$PATH CARGO_HOME=/Users/alexsong/.cache/cargo-home just test -p codex-core-skills
• git diff --cached --check
• just fmt attempted; Rust formatting completed, but the Python formatters could not download uncached Ruff wheels because files.pythonhosted.org is blocked in this sandbox.
• bazel mod deps --lockfile_mode=update/error --repo_env=ASPECT_TOOLS_TELEMETRY= --repo_env=DO_NOT_TRACK=1 evaluated the module graph and produced no MODULE.bazel.lock diff, but Bazel crashed on sandboxed sysctl during exit.

[alexsong-oai]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d8961dbcec

ℹ️ 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".

[chatgpt-codex-connector]

Commit the Bazel lockfile update

Because this adds a Rust workspace dependency, the Bazel module lock must be refreshed and committed; otherwise Bazel lock/check can fail in CI or for Bazel users until the just bazel-lock-update output is included, per repo guidance.

Useful? React with 👍 / 👎.

[alexsong-oai]

Ran just bazel-lock-update and just bazel-lock-check; both pass and MODULE.bazel.lock has no diff to commit.