Analysis: Point should not extend Kotlin Pair

[Copilot]

Issue requests analysis whether Point class should extend/use Kotlin's Pair. Analysis concludes: no implementation recommended.

Decision: Keep Point Independent

Point is a domain-specific 2D coordinate type with geometric semantics. Kotlin's Pair is a generic tuple. Merging them would:

• Lose semantic clarity: point.x/y → point.first/second
• Eliminate type safety: any Pair<Int, Int> becomes a valid Point
• Break 129 usage sites + 18 tests
• Require distance() as extension on all Pair<Int, Int> instances
• Provide zero functional benefit

Comparison

// Current: clear domain semantics
val point = Point(10, 20)
val dist = point.distance(other)  // x, y are self-documenting

// With Pair: loses meaning
typealias Point = Pair<Int, Int>
val point = Point(10, 20)
point.first  // Which coordinate is this?

val screenSize = Pair(1920, 1080)  // Same type, different concept
point.distance(screenSize)  // Type system cannot prevent this

Precedent

Doubleton<T, V> was retained during Kotlin migration rather than replaced with Pair due to domain-specific semantics (unordered equality, associated values). Same principle applies.

Kotlin Ecosystem Alignment

Standard library favors domain types over generic tuples:

• kotlin.time.Duration not Pair<Long, TimeUnit>
• kotlin.ranges.IntRange not Pair<Int, Int>
• androidx.compose.ui.geometry.Offset not Pair<Float, Float>

Documentation

• POINT_PAIR_ANALYSIS.md: Technical analysis with code examples, type safety implications, impact assessment
• IMPLEMENTATION_SUMMARY.md: Executive summary and resolution

No code changes. Current design is optimal.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Point to kotlin Pair</issue_title>
<issue_description>cz.vutbr.fit.interlockSim.util schould extends Pair from Tuples.kt.</issue_description>

<agent_instructions>analyse is good be and iplemement is possilbe</agent_instructions>

Comments on the Issue (you are @copilot in this section)

• Fixes Point to kotlin Pair #33

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[bedaHovorka]

Thank you for analyze, we can close this pr @copilot

[bedaHovorka]

impossible implementation