docs(github): record why an absent operation degrades to a no-op

Author: waleedlatif1

apps/sim/blocks/blocks/github.param-wiring.test.ts

@@ -219,3 +219,27 @@ describe('a stale advanced field cannot leak onto another operation', () => {
     }
   })
 })
+
+/**
+ * `providers/utils.ts` builds the provider `paramsTransform` input from
+ * `block.params` alone (`:776`), spreading `operation` in only for the
+ * tool-selection call (`:736-739`). Without an operation every alias skips —
+ * which must remain a no-op rather than a clobber, since that is exactly the
+ * behaviour that existed before this mapper.
+ */
+describe('absent operation degrades to a no-op, never a clobber', () => {
+  it.each([{}, { operation: undefined }, { operation: '' }, { operation: 123 }])(
+    'emits nothing for %o',
+    (opShape) => {
+      expect(
+        map({ ...opShape, milestone_title: 'x', reaction_content: 'heart', gist_public: 'true' })
+      ).toEqual({})
+    }
+  )
+
+  it('leaves every model-supplied value intact', () => {
+    const modelArgs = { content: 'rocket', title: 'model title', sort: 'newest', public: true }
+    const stale = { milestone_title: 'stale', fork_sort: 'oldest', gist_public: 'false' }
+    expect({ ...modelArgs, ...map({ ...modelArgs, ...stale }) }).toEqual(modelArgs)
+  })
+})

apps/sim/blocks/blocks/github.ts

@@ -2342,7 +2342,17 @@ Return ONLY the timestamp string - no explanations, no quotes, no extra text.`,
        * `providers/utils.ts` installs this as the provider `paramsTransform`,
        * spreading over the model's tool-call arguments — so emitting a key the
        * block did not supply would clobber a model-supplied value on the agent
-       * path, which is the one path these fields work on today.
+       * path.
+       *
+       * On the agent tool-calling path `operation` is not part of the params
+       * this receives: `providers/utils.ts` spreads it in for the tool-selection
+       * call (`:736-739`) but builds the transform's input from `block.params`
+       * alone (`:776`). Every alias therefore skips there, which is the same
+       * behaviour as before this mapper existed - the agent path already works
+       * because a model supplies `content`/`title`/`sort` by their real names.
+       * That gap is shared by every block whose mapper branches on
+       * `params.operation`, so closing it belongs in the provider layer rather
+       * than here.
        */
       params: (params) => {
         const result: Record<string, unknown> = {}