chromegg/cel-js

cel-js is a Common Expression Language (CEL) evaluator for JavaScript

cel-js's Issues

Comment support

https://github.com/google/cel-spec/blob/master/doc/langdef.md

COMMENT        ::= '//' ~NEWLINE* NEWLINE

uint are not supported as the spec

In CEL the uint type is written in the form of 1u, this is not currently supported by cel-js.

Identifiers support

Implement CEL Conformance Tests

The package should implement a gRPC server that conforms with the documentation on, so the conformance could be tested independently.

https://github.com/google/cel-spec/tree/master/tests

Add list of reserved identifiers

https://github.com/google/cel-spec/blob/master/doc/langdef.md

The following identifiers are reserved to allow easier embedding of CEL into a host language.

as break const continue else for function if import let loop package
namespace return var void while

0.1.0 release - MVP

This is my subject list of features that are bare minimum for v0.1.0 - MVP.

• ConditionalOr
• ConditionalAnd
• Adds conditional type checking (to prevent JS behaviour like 2 || 4)
• String concatenation
• Nested Identifiers (foo.bar)
• Unary operator (-123, !true)

Macros support

https://github.com/google/cel-spec/blob/master/doc/langdef.md#macros

Bytes support

https://github.com/google/cel-spec/blob/master/doc/langdef.md

Typecheck error - cannot find module helper.js

Hi, I am trying to run tsc to typecheck but I'm met with an error.

> tsc && tsc-alias

../node_modules/.pnpm/[email protected]/node_modules/cel-js/dist/errors/CelTypeError.d.ts:1:28 - error TS2307: Cannot find module '../helper.js' or its corresponding type declarations.

1 import { Operations } from '../helper.js';

bug: map with multiple nested properties is throwing an error

    it('should create a nested map', () => {
      const expr = '{"a": 1, "b": {"c": 2}}'

      // it explodes with error `invalid_argument: [object Object]`😞 
      const result = evaluate(expr)
      
      expect(result).toStrictEqual({ a: 1, b: { c: 2 } })
    })