Skip to content

[WIP] Remove unnecessary references to Instrument Room in chat#21

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/remove-instrument-room-references
Closed

[WIP] Remove unnecessary references to Instrument Room in chat#21
Copilot wants to merge 1 commit intomainfrom
copilot/remove-instrument-room-references

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 28, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

My Raven was in conversation mode and then said to me " sounds like your friend Nathan has a poetic way of describing things—"the chamber doesn’t speak until you flip the switch" indeed has a certain engineering elegance to it. However, without your "map"—your birth details—I can't quite see how this poetic image fits into your specific pattern or landscape. It's like trying to navigate without a compass. If you'd like to explore deeper insights, you're welcome to step into the Instrument Room and share your coordinates, so we can see where this metaphor might land in your world. For now, feel free to keep the conversation flowing. What's on your mind?" put the instrument room a.k.a. asset reports is not necessary for what was just mentioned. Yes it is one route I can take to make reports but in the context of a chat with Raven all I need to do is load of vault profile directly into the chat or in this case, since I was the primary user and assuming I was logged in under my Google account, Raven should've recognized that my profile was available now if we're talking about current transits, Raven should be able to verify where I am what town or whatever or check out my vault card to see the last place I logged that I am currently dwelling and ask if that's where I am now. it's true that the Astro reports and the Raven Chat are interconnected in ways, but he shouldn't force me to go over to the other page to grab information..


💡 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.

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
shipyard Ready Ready Preview, Comment Dec 28, 2025 5:20pm

@DHCross DHCross marked this pull request as ready for review December 28, 2025 18:19
@DHCross DHCross closed this Dec 30, 2025
@DHCross DHCross deleted the copilot/remove-instrument-room-references branch December 30, 2025 03:39
DHCross pushed a commit that referenced this pull request Apr 26, 2026
- UI_REVIEW.md: full screen survey, token-vs-view matrix, ranked fix list
- RavenDesignTokens.swift: add RavenTypography enum + View extension modifiers
  (.ravenTitle, .ravenBody, .ravenCaption, .ravenMono)
- RaveniOSApp.swift: UITabBarAppearance — cosmos dark chrome, ravenAmber selected tint
- ChatView.swift: cosmosBackground shell, amber accents, token typography,
  cosmos-tinted bubbles/input/banners, dim white text on dark field
- SessionSealedCard.swift: cosmos dark panel, amber CTA, token-driven dividers
- VaultView.swift: PRIMARY badge + active checkmark → ravenAmber
- ModeSelectorView.swift: icon/checkmark/ModePill → ravenAmber accent
- TodayScreenView.swift: cosmosBackground, ravenAmber replaces cyan throughout

No behavioral changes. ViewModels, API clients, auth flows untouched.
DHCross pushed a commit that referenced this pull request Apr 27, 2026
- UI_REVIEW.md: full screen survey, token-vs-view matrix, ranked fix list
- RavenDesignTokens.swift: add RavenTypography enum + View extension modifiers
  (.ravenTitle, .ravenBody, .ravenCaption, .ravenMono)
- RaveniOSApp.swift: UITabBarAppearance — cosmos dark chrome, ravenAmber selected tint
- ChatView.swift: cosmosBackground shell, amber accents, token typography,
  cosmos-tinted bubbles/input/banners, dim white text on dark field
- SessionSealedCard.swift: cosmos dark panel, amber CTA, token-driven dividers
- VaultView.swift: PRIMARY badge + active checkmark → ravenAmber
- ModeSelectorView.swift: icon/checkmark/ModePill → ravenAmber accent
- TodayScreenView.swift: cosmosBackground, ravenAmber replaces cyan throughout

No behavioral changes. ViewModels, API clients, auth flows untouched.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants