Fix undetected bug

src/Chanel-Tests/ChanelRemoveUnecesaryNotCleanerTest.class.st

@@ -164,6 +164,14 @@ ChanelRemoveUnecesaryNotCleanerTest >> testNotNot [
 	self assert: 'self test not not' isRewrittenAs: 'self test'
 ]
 
+{ #category : #tests }
+ChanelRemoveUnecesaryNotCleanerTest >> testRegression1 [
+	self denyIsRewritten: '| rewriter |
+	rewriter := self rewriter.
+	(self scope flatCollect: [ :class | class localMethods , class class localMethods ])
+		do: [ :method | (rewriter executeTree: method ast) ifTrue: [ method installAST ] ]'
+]
+
 { #category : #tests }
 ChanelRemoveUnecesaryNotCleanerTest >> testRejectNot [
 	self assert: '#() reject: [:e | e odd not ]' isRewrittenAs: '#() select: [:e |  e odd ]'

src/Chanel/RBVariableNode.extension.st

@@ -4,13 +4,15 @@ Extension { #name : #RBVariableNode }
 RBVariableNode >> canBeUselessNode [
 	"This is an ugly hack.
 	Pharo AST has a bug to know if a temporary is a declaration or a usage. See https://github.com/pharo-project/pharo/issues/6277
-	
+
 	This hack checks if the temporary is directly in its parent statements since declarations are not in the statements. In that case it's a candidate for removal."
+
+	self class environment at: #CopiedLocalVariable ifPresent: [ :c | (self variable isKindOf: c) ifTrue: [ ^ false ] ].
 	self isTempVariable ifTrue: [ ^ self parent statements includes: self ].
-	
+
 	"If the parent of an arg is a method node, then it's probably the argument definition."
-	self isArgumentVariable  ifTrue: [ ^ self parent isMethod not ].
-	
+	self isArgumentVariable ifTrue: [ ^ self parent isMethod not ].
+
 	^ super canBeUselessNode
 ]