Skip to content

Remove LLVM from infs#128

Merged
0xGeorgii merged 6 commits intomainfrom
126-update-infs-after-removing-llvm
Feb 18, 2026
Merged

Remove LLVM from infs#128
0xGeorgii merged 6 commits intomainfrom
126-update-infs-after-removing-llvm

Conversation

@0xGeorgii
Copy link
Contributor

Closes #126

@0xGeorgii 0xGeorgii added refactoring Project supplements infs Inference Start related labels Feb 16, 2026
@Inferara Inferara deleted a comment from codecov bot Feb 16, 2026
@Inferara Inferara deleted a comment from codecov bot Feb 16, 2026
@0xGeorgii 0xGeorgii self-assigned this Feb 16, 2026
@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

❌ Patch coverage is 90.71038% with 17 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
apps/infs/src/toolchain/conflict.rs 61.90% 8 Missing ⚠️
apps/infs/src/toolchain/doctor.rs 88.67% 6 Missing ⚠️
apps/infs/src/commands/self_cmd.rs 0.00% 2 Missing ⚠️
apps/infs/src/toolchain/resolver.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@0xGeorgii 0xGeorgii marked this pull request as ready for review February 16, 2026 09:11
@0xGeorgii 0xGeorgii requested a review from Copilot February 16, 2026 09:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the infs CLI and related docs/tests to reflect the removal of LLVM tooling, narrowing toolchain management and health checks to the infc compiler binary.

Changes:

  • Remove LLVM-related toolchain binaries/checks (inf-llc, rust-lld, libLLVM) and update doctor/TUI/test expectations accordingly.
  • Simplify toolchain path/symlink management to a single managed binary (infc) and update archive/scaffold/test utilities for the new layout.
  • Replace rand usage with fastrand and delete now-unused fixtures/helpers/APIs.

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
editors/vscode/QA_GUIDE.md Updates QA guidance/examples to match new toolchain warning scenarios.
apps/infs/tests/fixtures/type_error.inf Removes unused fixture.
apps/infs/tests/cli_integration.rs Updates doctor output assertions; removes redundant tests/helpers tied to removed fixtures/commands.
apps/infs/src/tui/widgets/mod.rs Updates widget module docs to match current widget set.
apps/infs/src/tui/views/doctor_view.rs Updates TUI doctor view tests to reflect infc-only checks.
apps/infs/src/tui/theme.rs Removes unused theme field.
apps/infs/src/toolchain/resolver.rs Switches to root-level infc resolution via binary_path.
apps/infs/src/toolchain/paths.rs Reduces managed binaries to infc, simplifies binary_path, and updates symlink/validation logic + tests.
apps/infs/src/toolchain/manifest.rs Removes unused manifest helper and test.
apps/infs/src/toolchain/download.rs Replaces rand jitter with fastrand.
apps/infs/src/toolchain/doctor.rs Removes LLVM checks and replaces generic binary checks with check_infc.
apps/infs/src/toolchain/conflict.rs Limits PATH conflict detection docs/logic to managed infc and updates tests.
apps/infs/src/toolchain/archive.rs Updates permission handling/tests to match infc root layout and removes LLVM artifacts from fixtures.
apps/infs/src/project/scaffold.rs Removes unused convenience API and switches temp dir randomness to fastrand.
apps/infs/src/project/mod.rs Removes unused re-exports tied to deleted APIs.
apps/infs/src/errors.rs Removes unused typed error variants, retaining only process-exit propagation.
apps/infs/src/commands/doctor.rs Updates command docs to reflect infc-only checks.
apps/infs/docs/qa-test-suite.md Updates test counts and fixture list; refreshes last-updated date.
apps/infs/README.md Updates docs to reflect only infc being symlinked.
apps/infs/Cargo.toml Swaps rand dependency for fastrand.
CHANGELOG.md Updates changelog wording to match infc-only symlinking.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0xGeorgii 0xGeorgii merged commit 73e9fd4 into main Feb 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infs Inference Start related refactoring Project supplements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update infs after removing LLVM

1 participant

Comments