fix: ignore implementationDetail hyps in rename_i (#5183)

Closes #5176

src/Lean/Elab/Tactic/BuiltinTactic.lean

@@ -458,6 +458,8 @@ def renameInaccessibles (mvarId : MVarId) (hs : TSyntaxArray ``binderIdent) : Ta
       match lctx.getAt? j with
       | none => pure ()
       | some localDecl =>
+        if localDecl.isImplementationDetail then
+          continue
         let inaccessible := !(extractMacroScopes localDecl.userName |>.equalScope callerScopes)
         let shadowed := found.contains localDecl.userName
         if inaccessible || shadowed then

tests/lean/run/5176.lean

@@ -0,0 +1,14 @@
+class Foo where
+
+class Bar extends Foo where
+  bar : True
+
+def foo : Foo := {}
+
+example [Bar] : Bar := {
+  foo with bar := by {
+    rename_i inst
+    guard_hyp inst : Bar
+    trivial
+  }
+}