Organize technical debt: map code comments to goals and issues

Reviewed all TODO, FIXME, EXTENSION comments in Kotlin codebase and categorized them for better tracking and prioritization. Comments now reference either long-term goals from LONG_TERM_GOALS.md or GitHub issues in the Backlog milestone.

Changes:
- Updated 11 code comments to reference specific goals (2, 4, 5, 15) or issues (#56-62)
- Created 6 GitHub issues for untracked technical debt (#57-62)
- Translated Czech comment in Train.kt about bidirectional operation
- Cleaned up unclear or outdated comment in MyResourceBoundle.kt

All changes are documentation-only with no functional impact.

Issues created:
#57 - Array2DMap NavigableMap support
#58 - Array2DMap modifiable EntrySet
#59 - HashMapGraph collection views performance
#60 - Track length validation (Goals 3, 4)
#61 - XML saveContext implementation (Goal 5)
#62 - Bidirectional train operation (Goals 3, 15)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: bedaHovorka

src/main/kotlin/cz/vutbr/fit/interlockSim/MyResourceBoundle.kt

@@ -29,7 +29,7 @@ class MyResourceBoundle private constructor() : ListResourceBundle() {
 	}
 
 	override fun getContents(): Array<Array<Any>>? {
-		// TODO Auto-generated method stub
+		// Returns null as resources are loaded via getFile() and getSchema() methods instead
 		return null
 	}
 

src/main/kotlin/cz/vutbr/fit/interlockSim/context/DefaultContext.kt

@@ -788,7 +788,7 @@ abstract class DefaultContext :
 			worker.terminate()
 		}
 		mainProcess?.terminate()
-		System.exit(1) // EXTENSION pryc
+		System.exit(1) // TODO: Remove System.exit - see GitHub issue #56
 	}
 
 	/**

src/main/kotlin/cz/vutbr/fit/interlockSim/sim/InOutWorker.kt

@@ -41,7 +41,9 @@ class InOutWorker(
 	private val pathFree: Condition =
 		object : Condition {
 			override fun test(): Boolean {
-				// EXTENSION az bude Interlocking: co kdyz cesta vubec neexistuje
+				// GOAL 2 & GOAL 4: Path finding and interlocking validation
+				// When automatic pathfinding is implemented (Goal 2), handle case where no valid path exists
+				// Related to interlocking validation (Goal 4) - see LONG_TERM_GOALS.md
 				// Local variable for smart cast since next is mutable property
 				val nextLocal = next
 				path = if (nextLocal != null) context.pathToNextSemaphore(inOut, nextLocal) else null

src/main/kotlin/cz/vutbr/fit/interlockSim/sim/Interlocking.kt

@@ -17,6 +17,8 @@ import cz.vutbr.fit.interlockSim.context.SimulationContext
 abstract class Interlocking(
 	protected val context: SimulationContext
 ) : LoopProcess() {
-	// EXTENSION hledani cest, Dikstruv algoritmus, move to control
+	// GOAL 2: Automatic Path Finding with Dijkstra algorithm
+	// Future: Implement automatic route calculation from entry to exit
+	// Priority: Critical | Estimate: 3 months | See LONG_TERM_GOALS.md Goal 2
 	// abstract fun setupWay(from: OrientedPathSeparator, to: OrientedPathSeparator)
 }

src/main/kotlin/cz/vutbr/fit/interlockSim/sim/Train.kt

@@ -73,9 +73,10 @@ class Train :
 			}
 		}.setFrequency(1.0)
 
-	// EXTENSION musi bud aspon umet prohodit zacatek a konec
-	// nebo je udalosti oba dva zrusit a dalsi udalosti obnovit, s tim ze vlak stoji na miste
-	// tj. "odpojeni" a "pripojeni" lokomotivy
+	// GitHub #62: Support bidirectional train operation (reverse direction)
+	// Allow train engineer to move to opposite end and drive in reverse direction.
+	// This is a simulation simplification of locomotive coupling/uncoupling operations.
+	// Implementation: Either swap start/end positions OR cancel/restore events with train stationary.
 
 	private abstract inner class Site : Process() { // lepsi nazev?
 		private val position: Variable = Variable(0.0)
@@ -188,7 +189,7 @@ class Train :
 			}
 			val path: Path? = context.pathToNextSemaphore(separator, next!!)
 
-			// EXTENSION stanice
+			// GOAL 15: Station stops for tutorial scenarios - see LONG_TERM_GOALS.md
 
 			if (semaphore.getSignal() == RailSemaphore.Signal.STOP) {
 				requireSimulation(getVelocity() >= 0) { "Velocity must be non-negative when approaching semaphore" }

src/main/kotlin/cz/vutbr/fit/interlockSim/util/Array2DMap.kt

@@ -23,7 +23,7 @@ import java.util.TreeSet
  * @param <V> type of values
  *
  */
-class Array2DMap<V> : AbstractMap<Point, V>() /* EXTENSION implements NavigableMap<Point, V> */ {
+class Array2DMap<V> : AbstractMap<Point, V>() /* Future: implements NavigableMap<Point, V> - see issue #57 */ {
 	private inner class Entry(
 		override val key: Point
 	) : MutableMap.MutableEntry<Point, V> {
@@ -194,7 +194,7 @@ class Array2DMap<V> : AbstractMap<Point, V>() /* EXTENSION implements NavigableM
 	fun getRow(y: Int): List<V> {
 		val list = array.get(y)
 
-		// EXTENSION zatim unmodifieable
+		// TODO: Make modifiable EntrySet - see issue #58
 		@Suppress("UNCHECKED_CAST")
 		val result: java.util.List<V> =
 			if (list == null) {

src/main/kotlin/cz/vutbr/fit/interlockSim/util/HashMapGraph.kt

@@ -159,7 +159,7 @@ class HashMapGraph<N, E, X> :
 	 */
 	override fun nodeSet(): Set<N> {
 		if (nodeCollection == null) nodeCollection = NodeCollection()
-		// EXTENSION pohled ne copii
+		// TODO: Return unmodifiable view instead of copy - see issue #59
 		val set = HashSet<N>(nodeCollection)
 		// for (Doubleton<N> c: map.keySet()) {
 		// 	set.addAll(c);

src/main/kotlin/cz/vutbr/fit/interlockSim/xml/XMLContextFactory.kt

@@ -59,7 +59,7 @@ import javax.xml.validation.Validator
 class XMLContextFactory :
 	EditingContextFactory,
 	SimulationContextFactory {
-	// EXTENSION co kdyz je delka koleje mezi InOuty mensi nez delka vlaku
+	// TODO: Validate track length >= train length - see issue #60 (relates to Goals 3 & 4)
 
 	private inner class XMLContext(
 		cols: Int,
@@ -329,7 +329,7 @@ class XMLContextFactory :
 		context: Context,
 		stream: OutputStream
 	): Boolean {
-		// TODO Auto-generated method stub
+		// TODO: Implement XML serialization - see issue #61 (relates to Goal 5)
 		return false
 	}