Plugin improvements (#132)

cmd/lint.go

@@ -16,8 +16,12 @@ import (
 type noopPluginCaller struct{}
 
 // CallContent implements evaluation.PluginCaller.
-func (n *noopPluginCaller) CallContent(ctx context.Context, name string, config evaluation.Configuration, invocation evaluation.Invocation, context plugin.MapData) (result string, diag diagnostics.Diag) {
-	return "", nil
+func (n *noopPluginCaller) CallContent(ctx context.Context, name string, config evaluation.Configuration, invocation evaluation.Invocation, context plugin.MapData) (result *plugin.ContentResult, diag diagnostics.Diag) {
+	return nil, nil
+}
+
+func (n *noopPluginCaller) ContentInvocationOrder(ctx context.Context, name string) (order plugin.InvocationOrder, diag diagnostics.Diag) {
+	return plugin.InvocationOrderUnspecified, nil
 }
 
 // CallData implements evaluation.PluginCaller.

cmd/render.go

@@ -15,7 +15,7 @@ import (
 	"github.com/blackstork-io/fabric/pkg/diagnostics"
 )
 
-func Render(ctx context.Context, blocks *parser.DefinedBlocks, pluginCaller *parser.Caller, docName string) (results []string, diags diagnostics.Diag) {
+func Render(ctx context.Context, blocks *parser.DefinedBlocks, pluginCaller *parser.Caller, docName string) (results string, diags diagnostics.Diag) {
 	doc, found := blocks.Documents[docName]
 	if !found {
 		diags.Add(
@@ -40,20 +40,13 @@ func Render(ctx context.Context, blocks *parser.DefinedBlocks, pluginCaller *par
 	return
 }
 
-func writeResults(dest io.Writer, results []string) (diags diagnostics.Diag) {
+func writeResults(dest io.Writer, results string) (diags diagnostics.Diag) {
 	if len(results) == 0 {
 		diags.Add("Empty output", "No content was produced")
 		return
 	}
 	w := bufio.NewWriter(dest)
-
-	// bufio.Writer preserves the first encountered error,
-	// so we're only cheking it once at flush
-	_, _ = w.WriteString(results[0])
-	for _, result := range results[1:] {
-		_, _ = w.WriteString("\n\n")
-		_, _ = w.WriteString(result)
-	}
+	w.WriteString(results)
 	_ = w.WriteByte('\n')
 	err := w.Flush()
 	diags.AppendErr(err, "Error while outputing result")

docs/plugins/builtin/content-providers/blockquote.md

@@ -0,0 +1,34 @@
+---
+title: blockquote 
+plugin:
+  name: blackstork/builtin
+  description: ""
+  tags: []
+  version: "v0.4.0"
+  source_github: "https://github.com/blackstork-io/fabric/tree/main/internal/builtin/"
+resource:
+  type: content-provider
+type: docs
+---
+
+{{< breadcrumbs 2 >}}
+
+{{< plugin-resource-header "blackstork/builtin" "builtin" "v0.4.0" "blockquote" "content provider" >}}
+
+The content provider is built-in, which means it's a part of `fabric` binary. It's available out-of-the-box, no installation required.
+
+
+#### Configuration
+
+The content provider doesn't support any configuration parameters.
+
+#### Usage
+
+The content provider supports the following execution parameters:
+
+```hcl
+content blockquote {
+    value = <string>  # required
+}
+```
+

docs/plugins/builtin/content-providers/code.md

@@ -0,0 +1,35 @@
+---
+title: code 
+plugin:
+  name: blackstork/builtin
+  description: ""
+  tags: []
+  version: "v0.4.0"
+  source_github: "https://github.com/blackstork-io/fabric/tree/main/internal/builtin/"
+resource:
+  type: content-provider
+type: docs
+---
+
+{{< breadcrumbs 2 >}}
+
+{{< plugin-resource-header "blackstork/builtin" "builtin" "v0.4.0" "code" "content provider" >}}
+
+The content provider is built-in, which means it's a part of `fabric` binary. It's available out-of-the-box, no installation required.
+
+
+#### Configuration
+
+The content provider doesn't support any configuration parameters.
+
+#### Usage
+
+The content provider supports the following execution parameters:
+
+```hcl
+content code {
+    language = <string>  # optional
+    value = <string>  # required
+}
+```
+

docs/plugins/builtin/content-providers/text.md

@@ -28,10 +28,7 @@ The content provider supports the following execution parameters:
 
 ```hcl
 content text {
-    absolute_title_size = <number>  # optional
-    code_language = <string>  # optional
-    format_as = <string>  # optional
-    text = <string>  # required
+    value = <string>  # required
 }
 ```
 

docs/plugins/builtin/content-providers/title.md

@@ -0,0 +1,36 @@
+---
+title: title 
+plugin:
+  name: blackstork/builtin
+  description: ""
+  tags: []
+  version: "v0.4.0"
+  source_github: "https://github.com/blackstork-io/fabric/tree/main/internal/builtin/"
+resource:
+  type: content-provider
+type: docs
+---
+
+{{< breadcrumbs 2 >}}
+
+{{< plugin-resource-header "blackstork/builtin" "builtin" "v0.4.0" "title" "content provider" >}}
+
+The content provider is built-in, which means it's a part of `fabric` binary. It's available out-of-the-box, no installation required.
+
+
+#### Configuration
+
+The content provider doesn't support any configuration parameters.
+
+#### Usage
+
+The content provider supports the following execution parameters:
+
+```hcl
+content title {
+    absolute_size = <number>  # optional
+    relative_size = <number>  # optional
+    value = <string>  # required
+}
+```
+