posit-dev / posit-dev/positron
`RSessionManager.getConsoleSession()` needs to learn about the multi-console world
Open
Nobody has claimed this yet.
lang: r
theme: multisession console
- Dominant language
- TypeScript
- Stars
- 4.3k
- Forks
- 184
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 206
Description
This currently is unaware of multi-console, and will always choose the newest created R session. That means if you:
- Start R 1
- Start and switch to R 2
- Switch back to R 1
Then IIUC getConsoleSession() will pick R 2 as the console session, which is not right.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with extensions/positron-r/src/session-manager.ts lines 150-181 and inspect how getConsoleSession() selects among R sessions. Reproduce the three-step sequence in the issue; done means switching back to R 1 causes getConsoleSession() to choose R 1 rather than the newest-created R 2.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r, typescript
- Domain
- desktop, tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100