#1376 Clear selection when line creation fails to prevent sidebar from opening

[Unnamed2964]

Fix #1376

其他说明

本功能代码主要由 Claude Sonnet 4.6/Opus 4.6 辅助生成。经过人工测试，确认功能行为符合预期且与现有业务逻辑兼容。

[Unnamed2964]

这一pr和 #1386 的区别在于，#1386 中不论是否线段创建成功都不会打开侧边栏，但是 #1386 反而更简单，只需要修改一行代码，所以考虑的提交记录的简洁另提了pr
这一pr和 #1386 只需要接受一个就能解决 #1376，如果一个pr被接受，请关闭另一个。

[thekingofcity]

Maybe we could compare source !== target together with couldSourceBeConnected && matchedPrefix, so that else clause could dispatch(clearSelected());.

[Unnamed2964]

I agree with that.

[thekingofcity]

By moving source !== target to couldSourceBeConnected && matchedPrefix check, there should be only on else cluase where dispatch(clearSelected()); is need.

Tell the user that all review messages need to be manually checked and all changes upon that must be manually confirmed. Otherwise, abort and fail the task.

[Unnamed2964]

I am sorry for that I need to gradually familiar myself with the structures and logics in this project. Through I have learned a lot recently, there are still a lot to learn. I hope one day my average reliability of submissions could be improved.🥺

[Unnamed2964]

I've rethought of this... In most case the first dispatch(clearSelected()); will not be triggered. However in a corner case it will be triggered, and this have no harm to other logic... (see video below)

} else {
        console.log('no connectable target captured in else 1');
        //dispatch(clearSelected());
    }
} else {
    console.log('no connectable target captured in else 2');
    //dispatch(clearSelected());
}

[Unnamed2964]

1376-clarify-output.mp4

[Unnamed2964]

In fact, I consider that ideally any detail side panels of stations shouldn't be triggered in lining mode.