Skip to content

Feat/api utilities#7

Merged
batuhanisildak-malwation merged 4 commits into
mainfrom
feat/api-utilities
Apr 29, 2026
Merged

Feat/api utilities#7
batuhanisildak-malwation merged 4 commits into
mainfrom
feat/api-utilities

Conversation

@ismailkoc-malwation

Copy link
Copy Markdown
Contributor

No description provided.

@batuhanisildak-malwation batuhanisildak-malwation left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Bunu localde nasil calistirip + claude'a baglayip test edebilecegimizi + runladiktan sonra mcpde bi sey degistirirsem eger claude'a onu nasil yukleyecegimi belirten seyleri contributing md acip yazar misin eger yoksa? MCP nasil maintain edilir ogrenelim, sslerle destekleyebilirsin. SSleri assets/'e koyalim.

Comment thread src/api/tools/submit.ts
Comment on lines +53 to +54
.optional()
.describe('MongoDB ObjectId of a network config from tz_network_configs_list'),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

MongoDB object id vs deme buna, network config id de dogrudan.

Comment thread src/api/tools/submit.ts
Comment on lines +49 to +50
preScript: z.string().optional(),
startArguments: z.string().optional(),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

su ikisine de desc, prescript sample oncesi vm'de calisacak commandi belirliyor. Start argument da sample'a arguman olarak eklenen seyler aslinda dumduz

Comment thread src/api/config.ts
Comment thread src/vnc/protocol/keysym-map.ts Outdated
Comment thread src/vnc/protocol/keysym-map.ts Outdated
Comment thread src/vnc/protocol/ra2ne-client.ts Outdated
Comment on lines +823 to +825
// extendedClipboard is required for clipboard writes to actually reach
// the host system clipboard on many VNC servers (notably the macOS one
// used in Threat.Zone sandboxes). See vencrypt-client.ts for details.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ai slop-ish comment, daha sade acikla

Comment thread src/vnc/protocol/vencrypt-client.ts Outdated
Comment on lines +441 to +447
// extendedClipboard is required for the modern UTF-8 clipboard sync path.
// Many VNC servers (notably the macOS one used in Threat.Zone sandboxes)
// only propagate clipboard writes to the host system clipboard when the
// Extended Clipboard message format is used; the legacy Latin-1 path is
// silently dropped. Advertising -260 here causes the server to send caps
// (handled in onServerCutText), which sets serverSupportsExtClipboard and
// switches clientCutText() to the deflate+UTF-8 path.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ai slop-ish comment, daha sade acikla

Comment thread src/vnc/tools/type-text.ts Outdated
Comment on lines +13 to +21
// Characters that require the Shift modifier on a standard US ANSI layout.
// Without the bracketed Shift_L press/release, VNC servers translating
// keysym → physical-key map the bare key (e.g. `!` → `1` on US layouts).
//
// EXCLUSIONS — `~ | " @ #` are deliberately omitted. These five sit on
// different physical keys between US and UK ANSI; bracketing them with
// Shift_L causes the US/UK swap (`~`↔`|`, `"`↔`@`) when the VM keymap
// disagrees with the host. Sending the bare keysym lets the RFB server
// resolve the keysym semantically and produce the right glyph.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ai slop-ish comment, daha sade acikla neden burda var yeterli

@batuhanisildak-malwation batuhanisildak-malwation merged commit 2aa67d4 into main Apr 29, 2026
2 checks passed
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