fix: Adding a missing data type for Time

[traut]

No description provided.

[traut]

There is a bug here, though. It looks like the encoding is not working correctly: the tests pass but github_gist publisher fails with

2024-12-15 15:56:05 INF Publishing the content command=render target=overview
runtime: goroutine stack exceeds 1000000000-byte limit
runtime: sp=0x14020d92360 stack=[0x14020d92000, 0x14040d92000]
fatal error: stack overflow

runtime stack:
runtime.throw({0x103b0dce8?, 0x1029c421c?})
        /opt/homebrew/opt/go/libexec/src/runtime/panic.go:1067 +0x38 fp=0x16daf2da0 sp=0x16daf2d70 pc=0x102a0a3c8
runtime.newstack()
        /opt/homebrew/opt/go/libexec/src/runtime/stack.go:1117 +0x478 fp=0x16daf2ed0 sp=0x16daf2da0 pc=0x1029ef018
runtime.morestack()
        /opt/homebrew/opt/go/libexec/src/runtime/asm_arm64.s:342 +0x70 fp=0x16daf2ed0 sp=0x16daf2ed0 pc=0x102a10260

goroutine 1 gp=0x140000021c0 m=12 mp=0x14000101808 [running]:
runtime.typeAssert(0x105013c00, 0x1046a5980)
        /opt/homebrew/opt/go/libexec/src/runtime/iface.go:467 +0x144 fp=0x14020d92360 sp=0x14020d92360 pc=0x1029a6f24
github.com/blackstork-io/fabric/plugin/pluginapi/v1.encodeData({0x10475dc70, 0x140019d2f60})
        /Users/traut/Work/fabric/plugin/pluginapi/v1/data_encoder.go:47 +0x880 fp=0x14020d925b0 sp=0x14020d92360 pc=0x10367e490
github.com/blackstork-io/fabric/plugin/pluginapi/v1.encodeData({0x10475dc70, 0x140019d2f48})
        /Users/traut/Work/fabric/plugin/pluginapi/v1/data_encoder.go:48 +0x8fc fp=0x14020d92800 sp=0x14020d925b0 pc=0x10367e50c
github.com/blackstork-io/fabric/plugin/pluginapi/v1.encodeData({0x10475dc70, 0x140019d2f30})
        /Users/traut/Work/fabric/plugin/pluginapi/v1/data_encoder.go:48 +0x8fc fp=0x14020d92a50 sp=0x14020d92800 pc=0x10367e50c
github.com/blackstork-io/fabric/plugin/pluginapi/v1.encodeData({0x10475dc70, 0x140019d2f18})
        /Users/traut/Work/fabric/plugin/pluginapi/v1/data_encoder.go:48 +0x8fc fp=0x14020d92ca0 sp=0x14020d92a50 pc=0x10367e50c
github.com/blackstork-io/fabric/plugin/pluginapi/v1.encodeData({0x10475dc70, 0x140019d2f00})
        /Users/traut/Work/fabric/plugin/pluginapi/v1/data_encoder.go:48 +0x8fc fp=0x14020d92ef0 sp=0x14020d92ca0 pc=0x10367e50c
...

UPDATE: fixed

[traut]

There is an issue with data command which I'm not sure how to solve yet:

$ cat ./test.yaml
foo: aaa
bar: bbb
since: 2024-12-02T00:00:00Z
until: 2024-12-08T23:59:59Z

$ cat ./test.fabric

document "test" {

  data yaml "test" {
    path = "./test.yaml"
  }

}

$ fabric data document.test.data.yaml.test
2024-12-20 11:44:39 WRN This is a dev version of the software! command=data version=0.4.2-dev+dirty.rev.bb58d06ad2950c5caab0a51cc085e34e3f56515b
2024-12-20 11:44:39 INF Parsing fabric files command=data directory=.
2024-12-20 11:44:39 INF Fetching data command=data target=document.test.data.yaml.test
2024-12-20 11:44:39 INF Loading document data command=data document=test datasource=yaml name=test
{
    "bar": "bbb",
    "foo": "aaa",
    "since": ,
    "until":
}

$

[dobarx]

LGTM. As for data cli command issue, it seems like there is a problem with library that is used to print this value:

github.com/TylerBrock/colorjson

fabric/cmd/data.go

Line 56 in 0dfddd6

fmt := colorjson.NewFormatter()

If you disable color while running data command it prints correctly:

fabric data document.test.data.yaml.test --color=false

BTW, I think maybe we should also rename flag to --no-color and flip value. Having bool default flags makes it requiring to pair them with =false.

[traut]

thanks for investigating the issue here, @dobarx!