Skip to content

First draft for an vscode extension api#191

Draft
jonah-iden wants to merge 1 commit intomainfrom
jiden/vscode-extension-api
Draft

First draft for an vscode extension api#191
jonah-iden wants to merge 1 commit intomainfrom
jiden/vscode-extension-api

Conversation

@jonah-iden
Copy link
Contributor

Adds an api to the vscode extension allowing other extensions to leverage the oct session to sync for example their own custom editors

Signed-off-by: Jonah Iden <jonah.iden@typefox.io>
Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

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

Looks good so far! How can you connect your custom editor with this API?

@@ -0,0 +1,189 @@
// ******************************************************************************
// Copyright 2024 TypeFox GmbH
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// Copyright 2024 TypeFox GmbH
// Copyright 2026 TypeFox GmbH

// ******************************************************************************

import * as vscode from 'vscode';
import { CollaborationInstance, PendingUser, PeerWithColor } from './collaboration-instance.js';
Copy link
Contributor

Choose a reason for hiding this comment

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

Shall we move the types that are made part of the API to this file (PendingUser, PeerWithColor), so we avoid breaking the API accidentally?

Copy link
Contributor

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Can we align the public API to be a drop-in replacement for the vsls package? We can expose more APIs in addition to that, but it would be great to make this a drop-in.

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.

3 participants