-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
Implement V8dotNet to make typescript-addin platform independent #5
Comments
In oder to use an extern TypeScript language service we delete all related files. we will then include the project CSharpTypescriptLanugageService or the related nugget. Reason: Why to exclude typescript? Observed in the past, it is like the the Typescript interface change. It is hard to identify which variable names, function calls was added or changed. See 1.3 -> 1.4 -> 1.5. - CSharpTypescriptLanugageService uses V8dotnet,Typescript and T4 templates to transpile the TypeScript interfaces/classes to C# interfaces/classes. - it also generates V8 alike function calls to access the V8 context. Delete the following file types: - TypeScript interface types which reflect "compiler/types.ts" and services/services.ts - delete helper classes used for marshaling return values of "JSON.stringify()". - delete javascript helper files like "completion.js". This commit shows also renamed files as deleted like: LanguageServiceShimHost to LanguageServiceHost which implements etc.
This commits add the typeScript language service base files. Alle future changes on the application are in favor off this files. - LanguageServiceHost.cs V8dotnet has two posibilites to access the v8 context. Annotating functions like Javascript.Net or using native function calle like v8 does it in c++. With this implementation we use the more native alike approche to gain performance. This prevent the usage of serializing/deserializing of javascript/managed objects. We will instead using reflection to get class members names and then access with these the members of an InternalHandle on the v8 context objects. V8LanguageServiceHostEnv will use the ILanguageServiceHost interface to call into managed site Host environment (LanguageServiceHost.cs). V8LanguageServiceHostEnv then marshals managed objects to V8 InternalHandle's to return them to the v8 context (and vice-versa). - V8TypescriptProvider.cs The native TypeScriptServices is statefull, this means we need to holde a reference to the code within the v8 context (called handles). To have a handle to the v8 context, we wrap the CSharpTypescriptLanguageService into a static class called V8TypescriptProvider.cs - TypeScriptContext.cs The TypeScriptContext was based on callback's to get return values from the native TypescriptService within the v8 context. With this changes we are able to use one function call to get return values from the v8 context. Since the TypeScriptContext.cs get destroyed after usage we use the V8TypescriptProvider to reinitialize it with state. Note: Even when we can use the V8TypescriptProvider every where (static), we should only use it within the TypeScriptContext. Furure changes are in order to implement this changes.
We use the native v8donet function call to access the v8 context. This prevents serializing/deserializing objects. We removed the depriecated function calls.
CsharpTypeScriptService transpiles ts classes and interfaces to c# classes in order to be compatible we can not extend this classes this means we need to implement adapters. e.g. NavigateToItemRegion is an adapter for NavigateToItem.
Change ICompileScriptOptions to ICompileOptions and implement new properties. Currently Projects tap does not work.
We remooved the initialization funktion, becouse the CsharpTypeScriptLanguageService is responisble for this.
Being able to use the TypeScript addin in MonoDevelop cross platform would be fantastic. So thanks for looking into this. I had a look at your fork of V8.NET and tried it on the Mac. It compiled but I could not get the console app to run. It was crashing with a |
You are right on this, saddly its a known issue, have a look at: Debugging the gives an exceptions at ObjectTemplateProxy::RegisterNamedPropertyHandlers What I did so far is using the clang AddressSanitizer, it seams that there are no issues there, means no memory leaks are detected. What I will do next is printing the Handle addresses out on the managed site and then use lldb to observe if I can find something. But in general I think v8dotnet will run in the future on Osx. |
Could you please have a look at the binaries that I have complied for OSX, seams to be working now. I also have created an addin for osx. I could not properly testing it from osx within vmware on an Ubuntu host because the GUI is not working fluently. Would be nice to have some feedback about the performance. I will also post some ideas tomorrow of the changes that I have made. |
Are there any plans to support Linux x86? Would compiling the V8.NET libraries for x86 work for both x86 and x64? |
I tried the V8.Net-Console.exe on the Mac and it looks good. Not tried the TypeScript addin with these binaries. The latest release you have seems to be 12 days old and does not seem to work. |
I currently build v8dotnet on x86 and updated the releases. On windows I build the typescript addin with the type Any CPU and added the x86 libraries of v8dotnet. On Linux it doesn't allow me to choose which kind of library I want to use. It always complains about:
Means I need to use the ia32.release build on x86 and x64.release on x64. I need to look further into this problem if there is a solution. |
@chrisber - Just tried my port of the TypeScript addin over to V8.NET with your V8.NET binaries on the Mac and it works. Great work 😄 |
This is nice hoping this will bring web development with Typescript+Angularjs and Asp vNext on Linux and Osx further. |
Done When
Using v8dotnet with the typescriptbinding
In this issue I will explain the assumptions, changes and thoughts that I made while investigating the support of v8dotnet for the typescript-binding. I will use FMC models to explain my thoughts. Feel free to correct me if I made a wrong assumption about something or share your opinion with me.
TypescriptBinding
I observed that the TypescriptBinding consists of the following building blocks: V8, Hosting, Provider and Project (MD) related belongings.
Providers
Provider seperate responisbilties of the Typescript language service (ts service api) The Typescript service API can be separated in different responsibilities like parsing, formatting, diagnostics, completion, info, compile. We implemented for each responsibilitiy one provider.
When it comes to compiling we have two possibilities. One solution is to use
getEmitOutput()
function which takes a filename returns the result, or we construct a compiler withgetProgram()
which takes a file and returns diagnostics. But then we have to implement an additional CompileHostEnvironment(). This is needed because the language service and the compiler using different host environments. The compiler uses:Compiler API , Host , EmitHost) , while the service uses TS API and Host.
- getEmitOutput()
- getProgram()
Note: On VS providers are executed on different threads threads
V8dotnet
V8dotnet is a wrapper around the chromium V8 project. Different then other similar projects like Javascript.Net it uses P/Invoke to focus on platform independence. V8dotnet is developed by James Wilkins and he is mostly the only maintainer of v8dotnet. V8dotnet has a similar syntax like the native V8 library. To start with v8donet I used the following resources.
An usage example of the of the v8dotnet library:
Native v8:
With V8dotnet:
Or an native approach:
Calling an object
CsharpTypeScriptLanguageService
The CsharpTypeScriptLanguageService excites of 3 projects.
TypeScriptLanguageServiceTranspiler
,TypeScriptLanguageService
andTypeScriptLanguageServiceTranspilerTest
,We will use the CsharpTypeScriptLanguageService to reflect the TypeScriptLanguageService API on the managed side.
While investigating V8dotnet and TypeScript and Monodevelop addin development. I recognized that the TypeScript API changes frequently. It is hard to observe which Types changed or what functionality was added. I decided to use TypeScript itself to generate classes/interfaces ( services.ts, types.ts ) that have c# syntax. This is done in the TypeScriptLanguageServiceTranspiler which is a rudimentary implementation to transpile typescript interfaces to c# classes. It also generates the TypeScriptLanguageService API from services.ts. The TypeScriptLanguageService uses an utility function to map the v8 context objects to managed objects. This is done with reflection in the TypeMapper() function.
Like Javascript.Net, v8dotnet is also able to use callbacks. We can just use an attribute on a function and v8donet is able to call the managed function. But this has the disadvantage that we need to annotate all classes which are part of the serialize and deserialize process (return and call parameter). One solution to avoid this problem is to use Jason.Stringify(), but serializing/deserializing is time consuming.
We have two possibilities to avoid serializing based on the same approach:
Static approach
This function constructs native call parameters and calls the getCompletionsAtPosition function of the Typescript language service. The return values are then marshaled into managed types. Here we are saving time because we don't need to serialize objects first.
To speed up development this can be implemented with an T4 template.
Second approach:
Since we already have classes we can use reflection to construct the object. The object obtains it's own properties and uses the handle to access the corresponding property in the v8 context. I read that reflection is also costly, when we need more performance we can abandon this approach and use the static generated approach, explained before.
There is also an issue with
Uninon types
, for instancevar foo = string | string[ ]
. This is easy we can just use string [ ] as property type and checking the length of the type to initializing it. But what can we do with thismessageText: string | DiagnosticMessageChain;
which comes from interface Diagnostics and DiagnosticMessageChain. Currently only the first type is selected.Host environment
We have two host environments the
managed host
and thev8 related host
.The managed host doesn't know anything about v8, while the
V8LanguageServiceHost
has an instance of the managedLangauageServiceHost
The
TypescriptContext
updates then theLanguageServiceHost
with new files and file infos. Furthermore, theV8LanguageServiceHost
calls theLanguageServiceHost
to get this information.IScriptSnapshotAdapter
This interface is used to exchange data between the managed LangauageServiceHost and the V8LanguageServiceHost to compute the get text change rate. Here we need an algorithm to calculate the exact line changes. Currently we search for the first line that changed.
CRLF vs LF
We need to make sure the all files have the same line endings before initializing the native typescript service. This option can not be changed afterwords.
Building the TypescriptBinding with CsharpTypeScriptLanguageService
git clone
git submodule update --init --recursive
Project
Monodevelop related stuff like addin-extensions for the project-options and monodevelop events.
Naming
There are a lot of service, typescript, shim host, parser names for objects and I introduced even more service, lanugeage ... I think it is good to have a naming convention.
And Maybe a directory structure.
Hosting
Provider
Binding // Extension points / gui related assets
What about namespaces?
Unit testing
For easier unit testing we can use SimpleInjector for IoC which should be fast by design and work on different platforms, but is it usable within Monodevelop?
Switch bettween the config for windows ( x86) , linux and osx (x86)
CsharpTypeScriptLanguageService has currently no .nupkg. To use it, we have included the CsharpTypeScriptLanguageService to the TypescriptBinding project.
Change the current project config to windows:
This libraries are costum builds of v8dotnet. V8dotnet uses an app start event to Load the dlls but this events get never called in mono.
contribution
This are some assumptions and ideas that I made while investigating v8dotnet for mono, if you like it would be nice if I can contribute something of this list to the project. We can start using gitter and hubord to keep track of things that needed to be done.
The text was updated successfully, but these errors were encountered: