Skip to content

Add Aliceserver as a debugger target. #440

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
dd86k wants to merge 12 commits into
base: master
Choose a base branch
from
Draft

Add Aliceserver as a debugger target. #440

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
bd2a5cc
mi2: Follow tsc type recommendations for some parameters
dd86k Aug 5, 2024
91c2249
Add Aliceserver as a test
dd86k Aug 5, 2024
eee2d29
Merge branch 'dd86k/lint-warnings' into dd86k/feature-aliceserver
dd86k Aug 7, 2024
bc62abe
Update Initial configuration
dd86k Aug 7, 2024
6fa6420
Add code to support Aliceserver
dd86k Aug 8, 2024
92e983a
package: Add printCalls and showDevDebugOutput for aliceserver
dd86k Aug 26, 2024
baa76e8
Misc. progress
dd86k Aug 26, 2024
5e7d193
Merge branch 'master' into dd86k/feature-aliceserver
dd86k Aug 26, 2024
f9e183f
package: forgot to set proper descriptions
dd86k Aug 28, 2024
e2e2a00
Progress on launching requests
dd86k Aug 28, 2024
f79eb59
mi2aliceserver: Redo MI2.stop() with a try-catch when killing debugge…
dd86k Aug 30, 2024
33dd9d9
Merge branch 'WebFreak001:master' into dd86k/feature-aliceserver
dd86k Feb 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/backend/mi2/mi2.ts
View file
Open in desktop
Copy link
Copy Markdown
Collaborator

@GitMensch GitMensch Feb 26, 2026

Choose a reason for hiding this comment

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

Can you please send a PR for that one to master?

I haven't checked the current related PR, but maybe there are other pieces outside of alice which you could include in the PR.

Thanks!

Copy link
Copy Markdown
Author

@dd86k dd86k Feb 26, 2026

Choose a reason for hiding this comment

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

I can make a PR for these changes. However, if I recall correctly, I did this commit following some tool's recommendation, forgot which, and I haven't checked if these changes cause additional issues. These alone do not depend on anything, thankfully.

Aliceserver & Alicedbg are still (super) WIP, so this PR series will be up as a draft for a while... Sorry.

Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ class LogMessage {
protected logReplaceTest = /{([^}]*)}/g;
public logMsgBrkList: Breakpoint[] = [];

logMsgOutput(record){
logMsgOutput(record: { isStream?: boolean; type: any; asyncClass?: string; output?: [string, any][]; content: any; }){
if ((record.type === 'console')) {
if(record.content.startsWith("$")){
const content = record.content;
Expand All @@ -54,7 +54,7 @@ class LogMessage {
}
}

logMsgProcess(parsed){
logMsgProcess(parsed: MINode){
this.logMsgBrkList.forEach((brk)=>{
if(parsed.outOfBandRecord[0].output[0][1] == "breakpoint-hit" && parsed.outOfBandRecord[0].output[2][1] == brk.id){
this.logMsgVar = brk?.logMessage;
Expand Down Expand Up @@ -626,7 +626,7 @@ export class MI2 extends EventEmitter implements IBackend {
return this.sendCommand("break-condition " + bkptNum + " " + condition);
}

setLogPoint(bkptNum, command): Thenable<any> {
setLogPoint(bkptNum: number, command: string): Thenable<any> {
const regex = /{([a-z0-9A-Z-_\.\>\&\*\[\]]*)}/gm;
let m:RegExpExecArray;
let commands:string = "";
Expand Down