feat: make stream's filter optional (#558)

When stream is defined and the template is not VE, if filter isn't set,
it'll error in build

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

packages/pages/docs/api/pages.stream.filter.md

@@ -9,7 +9,7 @@ The filter to apply to the stream
 **Signature:**
 
 ```typescript
-filter: {
+filter?: {
         entityIds?: string[];
         entityTypes?: string[];
         savedFilterIds?: string[];

packages/pages/docs/api/pages.stream.md

@@ -63,7 +63,7 @@ The fields to apply to the stream
 </td></tr>
 <tr><td>
 
-[filter](./pages.stream.filter.md)
+[filter?](./pages.stream.filter.md)
 
 </td><td>
 
@@ -73,7 +73,7 @@ The fields to apply to the stream
 
 </td><td>
 
-The filter to apply to the stream
+_(Optional)_ The filter to apply to the stream
 
 </td></tr>
 <tr><td>

packages/pages/etc/pages.api.md

@@ -289,7 +289,7 @@ export interface StaticTemplateConfig {
 export interface Stream {
   $id: string;
   fields: string[];
-  filter: {
+  filter?: {
     entityIds?: string[];
     entityTypes?: string[];
     savedFilterIds?: string[];

packages/pages/src/common/src/feature/stream.ts

@@ -1,5 +1,6 @@
 import { TemplateConfigInternal } from "../template/internal/types.js";
 import { RedirectConfigInternal } from "../redirect/internal/types.js";
+import { logErrorAndExit } from "../../../util/logError.js";
 
 /**
  * The shape of data that represents a stream configuration.
@@ -14,7 +15,7 @@ export interface StreamConfig {
   /** The fields to apply to the stream */
   fields: string[];
   /** The filter to apply to the stream */
-  filter: {
+  filter?: {
     /** The entity IDs to apply to the stream */
     entityIds?: string[];
     /** The entity types to apply to the stream */
@@ -48,6 +49,15 @@ export const convertTemplateConfigToStreamConfig = (
   if (!config) {
     return;
   }
+
+  if (
+    config.stream &&
+    !config.stream.filter &&
+    !config.additionalProperties?.isVETemplate
+  ) {
+    logErrorAndExit("Filter is required in StreamConfig for templates.");
+  }
+
   if (config.stream) {
     return {
       ...config.stream,

packages/pages/src/common/src/template/internal/types.ts

@@ -90,7 +90,7 @@ export interface StreamInternal {
   /** The fields to apply to the stream */
   fields: string[];
   /** The filter to apply to the stream */
-  filter: {
+  filter?: {
     /** The entity IDs to apply to the stream */
     entityIds?: string[];
     /** The entity types to apply to the stream */

packages/pages/src/common/src/template/types.ts

@@ -148,7 +148,7 @@ export interface Stream {
   /** The fields to apply to the stream */
   fields: string[];
   /** The filter to apply to the stream */
-  filter: {
+  filter?: {
     /** The entity IDs to apply to the stream */
     entityIds?: string[];
     /** The entity types to apply to the stream */