docs: record SGS hall POI visibility audit
This commit is contained in:
54
.superpowers/sdd/task-4-report.md
Normal file
54
.superpowers/sdd/task-4-report.md
Normal file
@@ -0,0 +1,54 @@
|
||||
# Task 4: Confirm floor-mode hall POI visibility — Implementation Report
|
||||
|
||||
## Status
|
||||
**DONE**
|
||||
|
||||
## Summary
|
||||
Source inspection confirms floor mode already prevents hall POI hiding via distance/limit pruning. No code change required.
|
||||
|
||||
## Findings
|
||||
|
||||
### Step 1: Floor visibility tier inspection
|
||||
Confirmed `getPoiVisibilityTier()` at `src/components/map/ThreeMap.vue:650-651`:
|
||||
```ts
|
||||
const getPoiVisibilityTier = (): PoiVisibilityTier => {
|
||||
if (activeView.value === 'floor') return 'full'
|
||||
```
|
||||
|
||||
### Step 2: Visibility tier behavior analysis
|
||||
When tier is `'full'`:
|
||||
- `getPoiVisibilityLimit('full')` returns `Number.POSITIVE_INFINITY`.
|
||||
- `getPoiScreenSpacing('full')` returns `0`.
|
||||
|
||||
### Step 3: Pruning mechanism verification
|
||||
In `updatePoiVisibilityByDistance()`:
|
||||
- `exceedsLimit` is false for `Infinity` because `Number.isFinite(Infinity)` is false.
|
||||
- `isTooClose` is false when spacing is 0.
|
||||
- `exhibition_hall` and `exhibition_hall_entrance` are in `corePoiCategories`.
|
||||
|
||||
### Conclusion
|
||||
Floor mode POI visibility is already correctly configured:
|
||||
1. Distance pruning is disabled.
|
||||
2. Count pruning is disabled.
|
||||
3. Hall POIs pass category checks.
|
||||
4. Selected/focused POIs always show.
|
||||
|
||||
No code modification needed.
|
||||
|
||||
## Files Inspected
|
||||
- `src/components/map/ThreeMap.vue`
|
||||
|
||||
## Commands Run
|
||||
None required — source inspection confirms correct behavior.
|
||||
|
||||
## Commits Created
|
||||
None — no code changes.
|
||||
|
||||
## Self-Review
|
||||
- Floor mode already returns `'full'` visibility tier.
|
||||
- `'full'` tier prevents both distance and count pruning.
|
||||
- Hall POIs are included in `corePoiCategories`.
|
||||
- Logic is correct and no defensive guard is needed.
|
||||
|
||||
## Concerns
|
||||
None.
|
||||
Reference in New Issue
Block a user