Improve `E0603` suggestions for private grouped imports by raushan728 · Pull Request #158445 · rust-lang/rust

raushan728 · 2026-06-26T12:49:34Z

After #156244 removed broken suggestions for nested imports, E0603 was left without any suggestion for grouped use statements.

This PR restores those suggestions - it splits the failing item out of the group into a corrected use statement, replaces the whole statement if the group would become empty, and handles re-export chains.

Fixes #157453.

rustbot · 2026-06-26T12:49:39Z

r? @jackh726

rustbot has assigned @jackh726.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

Owners of files modified in this PR: compiler
compiler expanded to 73 candidates
Random selection from 18 candidates

petrochenkov · 2026-06-29T14:54:14Z

I reviewed everything except the compiler/rustc_resolve/src/error_helper.rs part, which is full of byte pos arithmetics and other wonders.

r? @fee1-dead please review, since you wanted this.

rustbot · 2026-06-29T14:54:19Z

fee1-dead is not on the review rotation at the moment.
They may take a while to respond.

rustbot · 2026-07-09T20:16:20Z

Reminder, once the PR becomes ready for a review, use @rustbot ready.

raushan728 · 2026-07-10T13:51:31Z

@rustbot ready

fee1-dead

A lot of these string fiddling is still not sitting well with me. It shouldn't be too hard to use the string search functions Rust already have to do some of these. Please use them, as we want compiler code to be readable.

View changes since this review

oli-obk · 2026-08-04T16:14:04Z

I invite you to join https://rust-lang.zulipchat.com/join/rlfvpemsaacs3pfi6kwqnqjb/ and start a thread asking for a mentor if you want to make progress here.

cjgillot · 2026-08-04T18:45:17Z

+                    if !inner.contains(',') && !inner.is_empty() {
+                        replacement = format!("{}{}", replacement[..open].trim_end(), inner);
+                    }
+                }


Don't try to be smart with braces. The user has rustfmt.

View changes since the review

I wasn't sure I understood this comment my intention wasn't just formatting but to produce a cleaner suggestion (use foo::Bar instead of use foo::{Bar}) when only one item remains. does rustfmt rewrite use foo::{Bar} into use foo::Bar, or is the idea that we shouldn't try to simplify the syntax in diagnostics?

we shouldn't try to simplify the syntax in diagnostics?

i dont thing the UI should change regardless of what we do in the code.

does rustfmt rewrite use foo::{Bar} into use foo::Bar, or is the idea that we shouldn't try to simplify the syntax in diagnostics?

Both.

Simplifying the syntax is ok when it makes rustc code simpler too. If you need to count BytePos for span arithmetic, it's not worth it. (Span arithmetic is known to cause ICEs when the user uses multibyte characters.)

As far i can tell report_privacy_error only has root_span and span_to_remove not the UseTree AST, so I couldn't find existing helper that would simplify {Bar} to Bar without doing manual BytePos-based span arithmetic. check_unused.rs can do this because it still has the UseTree spans available.

i wanted the diagnostic to present the import in the form users would normally write use foo::Bar rather than use foo::{Bar} especially for a single remaining item. That said, if the extra span arithmetic isn't considered worth the complexity, i'm happy to drop brace simplification but, personally i dont want we should suggest {Bar} instead of Bar for a single item.

What should i do here should i remove?

I just removed.

cjgillot · 2026-08-04T18:46:34Z

+                // Replace the entire `use` instead of leaving `use foo::{}`.
+                if leaves_empty_group {
+                    let line_span = self.tcx.sess.source_map().span_extend_to_line(root_span);
+                    let suggestion_text = format!("{indentation}use {path};");
+                    err.multipart_suggestion(
+                        msg,
+                        vec![(line_span, suggestion_text)],
+                        Applicability::MachineApplicable,
+                    );
+                    break;
+                }


Why not just replace the contents of root_span with path?

View changes since the review

That's what the current version does now. In the single-item case I replace root_span with path which reuses the existing use keyword instead of replacing the whole statement.

cjgillot · 2026-08-04T18:48:23Z

+                // Insert before `root_span` to reuse the existing `use`.
+                err.multipart_suggestion(
+                    msg,
+                    vec![
+                        (root_span.shrink_to_lo(), format!("{path};\n{indentation}use ")),


Would it be simpler to insert a clean line at line_span.shrink_to_lo()?

View changes since the review

I considered that, but I chose root_span.shrink_to_lo so the suggestion only depends on the import path span and preserves formatting when the use statement isn't alone on its line (for example inside inline blocks).

Suggest direct imports for private items inside grouped imports. Split grouped imports when necessary and replace single-item groups with direct imports.

rustbot · 2026-09-27T14:19:39Z

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

raushan728 · 2026-10-01T13:09:20Z

@rustbot reroll

folkertdev · 2026-10-04T12:39:05Z

I definitely don't have context here, @cjgillot @fee1-dead @petrochenkov can one of you pick this up?

raushan728

Please do not review for now

View changes since this review

rustbot assigned jackh726 Jun 26, 2026

rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 26, 2026

rustbot assigned petrochenkov and unassigned jackh726 Jun 26, 2026

rustbot assigned fee1-dead and unassigned petrochenkov Jun 29, 2026

raushan728 mentioned this pull request Jun 30, 2026

Improve E0277 diagnostics for conditionally implemented traits #158494

Merged

fee1-dead requested changes Jul 9, 2026

View reviewed changes

rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 9, 2026

raushan728 force-pushed the issues/157453 branch from 59ad765 to 4706b08 Compare July 10, 2026 13:37

This comment has been minimized.

Sign in to view

rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 10, 2026

This comment has been minimized.

Sign in to view

raushan728 force-pushed the issues/157453 branch from 4706b08 to 12d8c86 Compare July 13, 2026 15:20

This comment has been minimized.

Sign in to view

raushan728 requested a review from fee1-dead July 15, 2026 11:04

fee1-dead requested changes Jul 22, 2026

View reviewed changes

rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 22, 2026

raushan728 force-pushed the issues/157453 branch from 12d8c86 to 567fb3a Compare July 25, 2026 13:18

This comment has been minimized.

Sign in to view

rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Aug 2, 2026

fee1-dead added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 2, 2026

oli-obk unassigned fee1-dead Aug 4, 2026

cjgillot reviewed Aug 4, 2026

View reviewed changes

cjgillot self-assigned this Aug 5, 2026

raushan728 force-pushed the issues/157453 branch from 0c6885b to e8eb880 Compare August 5, 2026 09:07

This comment has been minimized.

Sign in to view

raushan728 force-pushed the issues/157453 branch from e8eb880 to fb597c3 Compare August 5, 2026 09:38

raushan728 requested a review from cjgillot August 5, 2026 09:45

rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 5, 2026

raushan728 added 2 commits September 27, 2026 11:04

test(imports): add baseline test for E0603 grouped import suggestion

7bfc781

fix(resolve): improve E0603 suggestions for grouped imports

601c41c

Suggest direct imports for private items inside grouped imports. Split grouped imports when necessary and replace single-item groups with direct imports.

raushan728 force-pushed the issues/157453 branch from fb597c3 to 601c41c Compare September 27, 2026 14:19

rustbot assigned BoxyUwU and unassigned cjgillot Oct 1, 2026

rustbot assigned folkertdev and unassigned BoxyUwU Oct 1, 2026

raushan728 commented Oct 4, 2026

View reviewed changes

raushan728 marked this pull request as draft October 4, 2026 13:55

rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 4, 2026

raushan728 closed this Oct 4, 2026

rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Oct 4, 2026

raushan728 deleted the issues/157453 branch October 4, 2026 17:26

Conversation

raushan728 commented Jun 26, 2026 • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rustbot commented Jun 26, 2026

Uh oh!

petrochenkov commented Jun 29, 2026

Uh oh!

rustbot commented Jun 29, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rustbot commented Jul 9, 2026

Uh oh!

This comment has been minimized.

raushan728 commented Jul 10, 2026

Uh oh!

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

fee1-dead left a comment • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment has been minimized.

oli-obk commented Aug 4, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cjgillot Aug 4, 2026 • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cjgillot Aug 4, 2026 • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cjgillot Aug 4, 2026 • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

This comment has been minimized.

rustbot commented Sep 27, 2026

Uh oh!

raushan728 commented Oct 1, 2026

Uh oh!

folkertdev commented Oct 4, 2026

Uh oh!

raushan728 left a comment • edited by rustbot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

raushan728 commented Jun 26, 2026 •

edited by rustbot

Loading

fee1-dead left a comment •

edited by rustbot

Loading

cjgillot Aug 4, 2026 •

edited by rustbot

Loading

cjgillot Aug 4, 2026 •

edited by rustbot

Loading

cjgillot Aug 4, 2026 •

edited by rustbot

Loading

raushan728 left a comment •

edited by rustbot

Loading