"main" entry in package.json should be: "main": "dist/pinets.min.es.js"

[dcaoyuan]

"main" entry in package.json should be:
"main": "dist/pinets.min.es.js"

instead of:
"main": "dist/pinets.dev.es.js"

[dcaoyuan]

Let me see how to config my project to use 'pinets.min.browser.js' or 'pinets.dev.browser.js'

[alaa-eddine]

Hi @dcaoyuan you're right I need to update the package.json.

regarding the browser build, you can download the latest one from here https://github.com/QuantForgeOrg/PineTS/releases/tag/v0.5.0

[alaa-eddine]

Hey @dcaoyuan going forward you can load PineTS in the browser using jsdelivr CDN :)

<script src="https://cdn.jsdelivr.net/npm/pinets/dist/pinets.min.browser.js"></script>

[dcaoyuan]

I'm going to add 'PineTS' as a dependency package in my project:
https://github.com/dcaoyuan/vibetrader

I did a little bit quick and dirty modification of PineTS to make it as a local dependency:
dcaoyuan@90f8f95

Looking forward to a better solution.

[alaa-eddine]

@dcaoyuan hmm I see, I'll check your case and try to provide a better solution.
Your project looks cool , seems like we're having the same idea :D I just released QFChart a few days ago, that will be the visualization companion library for PineTS https://quantforgeorg.github.io/QFChart/demos/full.html

and I'm preparing the next release for PineTS which will introduce the possibility to run pinescripts directly , this will be experimental, but I tested it and it works pretty good as long as you don't use unimplemented methods .

the PineTS output is tweaked for QFChart to make the integration seamless

[alaa-eddine]

@dcaoyuan question regarding your issue.
did you try using the browser script that I shared above ?
or using import syntax if you're using <script type="module"> ?

[dcaoyuan]

I import it via:
https://github.com/dcaoyuan/vibetrader/blob/main/package.json#L15

without my modification, I will get error during npm run build:

 Build failed in 3.94s
error during build:
node_modules/pinets/dist/pinets.dev.es.js (23:9): "fileURLToPath" is not exported by "__vite-browser-external", imported by "node_modules/pinets/dist/pinets.dev.es.js".
file: /home/dcaoyuan/myworks/vibetrader/node_modules/pinets/dist/pinets.dev.es.js:23:9

21: import * as fs from 'fs';
22: import * as path from 'path';
23: import { fileURLToPath } from 'url';
             ^
24: 
25: var __defProp$d = Object.defineProperty;

    at getRollupError (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/parseAst.js:401:41)
    at error (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/parseAst.js:397:42)
    at Module.error (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:16946:16)
    at Module.traceVariable (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:17400:29)
    at ModuleScope.findVariable (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:15067:39)
    at Identifier.bind (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:5415:40)
    at CallExpression.bind (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:2806:23)
    at CallExpression.bind (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:12114:15)
    at VariableDeclarator.bind (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:2806:23)
    at VariableDeclaration.bind (file:///home/dcaoyuan/myworks/vibetrader/node_modules/rollup/dist/es/shared/node-entry.js:2802:28)

Not sure how to import it as 'browser' module automatically.

[alaa-eddine]

Ok I see the problem.
the import statement is using the nodejs version which is using fileURLToPath
I will try to address this in the next release. I'll probably push it tomorrow .

[alaa-eddine]

hey @dcaoyuan I just released the new version of PineTS (v0.7.0) this one comes with an updated build pipeline that handles multiple bundles.
please check it out and let me know if it works for you natively

[dcaoyuan]

@alaa-eddine Yes, I can confirm that it works now.

But, 'IProvider' is not exported anymore? although I can use 'any[]' as the data provider.

[alaa-eddine]

IProvider is an interface so it does not translate to a JS object, however, it's present as a type in the distribution package.
if you can't see it it might just means that your project is not using the types exposed by the package.

I can see the types properly exposed

and I also see IProvider type in the npmjs package

[dcaoyuan]

It's strange, I cannot get following code built:

import type { Kline } from "./Kline";
import type { TVar } from "../timeseris/TVar";
import type { IProvider } from "pinets/dist/types/marketData/IProvider"


export class TSerProvider implements IProvider {
    data: Kline[];

    constructor(kvar: TVar<Kline>) {
        this.data = kvar.toArray();
    }

    async getMarketData(tickerId: string, timeframe: string, limit?: number, sDate?: number, eDate?: number): Promise<unknown> {
        return this.data;
    }
}

npm run build will output:

> vibetrader@0.0.1 build
> tsc -b && vite build

src/lib/domain/TSerProvider.ts:3:32 - error TS2307: Cannot find module 'pinets/dist/types/marketData/IProvider' or its corresponding type declarations.

3 import type { IProvider } from "pinets/dist/types/marketData/IProvider"
                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


Found 1 error.

I'll try to find out why.

BTW, not sure if it's better to also export IProvider here:

PineTS/src/index.ts

Line 8 in fb645f8

export { PineTS, Context, Provider };

[alaa-eddine]

you're probably right, I'll need to export it explicitly here, or generate a barrel file with the whole index.

as a workaround you can copy just the IProvider.ts in your code, it's just type and interface definition it does not generate any JS code.

as long as your provider exposes the two expected methods (getMarketData and getSymbolInfo) it should be fine, you can register it.
even if you don't explicitly declare it with "implements IProvider" part.

I'll do more tests with the exposed types and fix it in the next build

[alaa-eddine]

I just noticed that the latest npm package did not contain the right bundle files,
I published a new pacage, please make sure to use v0.7.1