Wood sitter #53
Wood sitter #53
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9d3578222b
ℹ️ 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".
fuck you and the person who does a poor job of maintaining this
|
@codex pls review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c6b29bae5c
ℹ️ 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".
|
@codex pls review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c87e8cc36e
ℹ️ 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".
|
@codex pls review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bff1b10605
ℹ️ 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".
|
@codex pls review |
|
Codex Review: Didn't find any major issues. Keep them coming! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Collect unresolved import samples even when graph output is disabled and return them from resolveImportLinks. Emit bounded unresolved import sample lines in postScanImports alongside the aggregate import summary counts.
Workflows now include patch files in node_modules cache keys and skip npm run patch on cache hits while still rebuilding native modules. Replaced the tree-sitter patch with a minimal source-only binding.gyp diff to avoid cross-platform patch-package failures. Upgraded scheduler native smoke/language tests to a shared per-language metadata contract suite covering fixture routing and chunk metadata invariants.
Removed the fixed 8s hover timeout cap in collectLspTypes and made hover timeout configurable per provider. Added sourcekit-specific timeout/retry/breaker/hover controls with safer defaults and enabled passthrough of tooling.sourcekit/pyright config in getToolingConfig. Validated with sourcekit provider fallback/output-shape and LSP enrichment tests.
- add hover throttling, adaptive timeout disable, per-file budgets, and metrics in LSP collector - decouple SourceKit hover timeout, prefer non-asserts binary resolution, and add host-level concurrency gate for test runs - improve Swift signature parsing coverage to reduce hover dependence and add focused parser tests - include atomic JSON stream replace-path hardening updates in src/shared/json-stream/atomic.js
|
@codex pls review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b8d9328e85
ℹ️ 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".
1 participant
Summary
Checklist