Skip to content

Conversation

@MarkDuckworth
Copy link
Contributor

@MarkDuckworth MarkDuckworth commented Jul 18, 2025

Changes:

  1. Add expressions: arrayReverse, ceil, exp, floor, countDistinct, pow, collectionId, length, ln, log, round, sqrt, strReverse
  2. Remove expressions: replaceFirst, replaceAll, bitAnd, bitRightShift, bitLeftShift, bitXor, bitOr, bitNot
  3. Rename expressions: arrayOffset -> arrayGet, substr -> substring
  4. Fix QualifiedResourcePath parsing. Issues discovered during expression implementation.
  5. API Review - renames and documentation fixes.

Missing expressions: concat, currentTimestamp, is, log10, mask, timestampTrunc

…Also include @google-cloud/firestore/pipelines as an alternative submodule import
…gleapis/nodejs-firestore into markduckworth/pipeline-options
…gleapis/nodejs-firestore into markduckworth/pipeline-options
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/nodejs-firestore API. label Jul 18, 2025
MarkDuckworth and others added 6 commits July 21, 2025 09:11
* feat: Add strReverse expression

* Update order.ts

revert jules' changes to compare ut8 strings test

---------

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
* Fix reference parsing

* Throw on parsing a reference to a collection or db root, until we define the SDK behavior.

* Address pr feedback and other cleanup
…googleapis/nodejs-firestore into markduckworth/pipelines-new-expressions
…re into markduckworth/pipelines-new-expressions
Base automatically changed from markduckworth/pipeline-errors to feat/pipelines July 31, 2025 18:13
@MarkDuckworth MarkDuckworth marked this pull request as ready for review July 31, 2025 18:19
@MarkDuckworth MarkDuckworth requested review from a team as code owners July 31, 2025 18:19
@MarkDuckworth MarkDuckworth changed the title Markduckworth/pipelines new expressions More expressions Jul 31, 2025
* Renamed XWithAlias to AliasedX

* API cleanup

* rename generic to raw

* API review - PipelineResult.data() should never return undefined

* customOptions to rawOptions

* custom to raw options

* code cleanup

* Fixes on reaname

* API review fixes: PipelineExecuteOptions

* API review: PipelineResult

* Rename args to divide as dividend and divisor

* Rename params to subtract

* API audit

* Remove @beta tag

* Doc review start

* enable all tests

* TSDoc todos

* Script/simple agent for updating tsdoc

* Pipeline stream tests
@MarkDuckworth MarkDuckworth changed the title More expressions More expressions and API cleanup Aug 19, 2025
@wu-hui wu-hui assigned MarkDuckworth and unassigned wu-hui Aug 19, 2025
@MarkDuckworth MarkDuckworth merged commit 689a03d into feat/pipelines Oct 21, 2025
7 of 12 checks passed
@MarkDuckworth MarkDuckworth deleted the markduckworth/pipelines-new-expressions branch October 21, 2025 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the googleapis/nodejs-firestore API. size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants