Skip to content

Move thermal implant to med research - #5545

Merged
Rinary1 merged 3 commits into
ss14Starlight:starlight-devfrom
Littlemen07:thermal
Aug 16, 2026
Merged

Move thermal implant to med research#5545
Rinary1 merged 3 commits into
ss14Starlight:starlight-devfrom
Littlemen07:thermal

Conversation

@Littlemen07

@Littlemen07 Littlemen07 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Short description

Move thermal implant to med tech, move night vision to tier 2 since thermal are tier 3

Why we need to add this

After a round where the entiretity of sec couldnt catch a single umbrea that was always invisible, i realised i almost never saw thermal implant in thousands of hours.

Thats because they are an arnenal tech, a tier 3 one. Making it never seen even on warop.

This PR changes moves it to medical, which is even the place where you got to print those in the first place. Allowing for three departement to cooperate (science, med and sec) in order to be able to counter invisible threats, instead of just existing in a tech tree that isnt usually touched and being unreachable even when needed since arsenal is teched last.

Media (Video/Screenshots)

none

Checks

  • I do not require assistance to complete the PR.
  • Before posting/requesting review of a PR, I have verified that the changes work.
  • I have added screenshots/videos of the changes, or this PR does not change in-game mechanics.
  • I affirm that my changes are licensed under the MIT License and grant permission for use in this repository under its conditions.

Changelog
🆑 Romero

  • tweak: Thermal implant are now a tech 3 med technology.
  • tweak: NVG are moved to tier 2.

@Littlemen07
Littlemen07 requested a review from a team August 6, 2026 16:43
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c3eb4897-b84c-4c05-a96b-9708b02f24bd

📥 Commits

Reviewing files that changed from the base of the PR and between 5779393 and a034733.

📒 Files selected for processing (1)
  • Resources/Prototypes/_Starlight/Research/medical.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • Resources/Prototypes/_Starlight/Research/medical.yml

Included review availability: Your plan includes up to 2 reviews per rolling hour; 1 remains after this review.


📝 Walkthrough

Summary

Moves CyberEyeThermalVision from tier-3 Arsenal technology to tier-3 Biochemical medical technology.

Moves CyberEyeNightVision from tier 3 to tier 2.

This enables cooperation between science, medical, and security departments against invisible threats. Thermal cyber-eyes now require CyberEyeNightVision.

Notable changes

  • Removes the thermal cyber-eye technology and related declarations from arsenal.yml.
  • Adds tier-3 CyberEyeThermalVision to medical.yml.
  • Updates CyberEyeNightVision to tier 2.

Risk areas

  • Research progression changes for departments that previously accessed thermal cyber-eyes through Arsenal technology.
  • Technology prerequisites and unlock references must remain valid.
  • Medical research now owns the thermal cyber-eye unlock and its associated progression.

Guideline gaps

  • The PR context does not confirm Shared-first placement.
  • The PR context does not confirm Starlight ownership comments.
  • No localization changes are shown.
  • The PR context does not describe breaking-change handling.
  • The PR context does not describe broader design impact beyond improved access to thermal implants.

Walkthrough

The research tree moves CyberEyeNightVision to tier 2 and adds CyberEyeThermalVision as a tier-3 Biochemical technology. The former Arsenal declaration is removed.

Changes

Cyber-eye research

Layer / File(s) Summary
Research tier and thermal technology definitions
Resources/Prototypes/_Starlight/Research/medical.yml, Resources/Prototypes/_Starlight/Research/arsenal.yml
CyberEyeNightVision moves from tier 3 to tier 2. CyberEyeThermalVision is added to the Biochemical discipline with its prerequisite, recipe, icon, and Security/Medical radio channels. The previous Arsenal declaration is removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to a0347

This PR changes the research placement of thermal implants and night vision. No actionable merge-blocking risk remains based on the supplied evidence.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the primary change: moving the thermal implant to medical research.
Description check ✅ Passed The description explains the research-tree changes and the goal of improving access to thermal implants.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/S S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Aug 6, 2026
@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

@Rinary1 Rinary1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

starlight... in starlight?

Comment thread Resources/Prototypes/_Starlight/Research/medical.yml Outdated
Comment thread Resources/Prototypes/_Starlight/Research/medical.yml Outdated
Comment thread Resources/Prototypes/_Starlight/Research/medical.yml Outdated
@starlightgithub starlightgithub Bot added S: Awaiting Changes Status: Changes are required before another review can happen and removed S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. labels Aug 11, 2026
Co-authored-by: Rinary <rinary.super@gmail.com>
@Rinary1
Rinary1 enabled auto-merge August 16, 2026 09:30
@starlightgithub starlightgithub Bot added S: Approved Status: Reviewed and approved by at least one maintainer or dev; a PR may require another approval. and removed S: Awaiting Changes Status: Changes are required before another review can happen labels Aug 16, 2026
@Rinary1
Rinary1 added this pull request to the merge queue Aug 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Content + Integration Test Results

Tests 📝 Passed ✅ Failed ❌ Duration ⏱️
1690 1671 0 12m 54s

🎉 No failed tests in this run.

Github Test Reporter by CTRF 💚

Merged via the queue into ss14Starlight:starlight-dev with commit 89960cd Aug 16, 2026
19 checks passed
starlightgithub Bot added a commit that referenced this pull request Aug 16, 2026
@coderabbitai coderabbitai Bot mentioned this pull request Aug 16, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: No C# Changes: Prototypes S: Approved Status: Reviewed and approved by at least one maintainer or dev; a PR may require another approval. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants