Skip to content

Bump Laya: source b9fe351c, model 7b928d82 - #611

Merged
bong-water-water-bong merged 3 commits into
mainfrom
bump-laya/b9fe351c-7b928d82
Oct 28, 2026
Merged

bong-water-water-bong merged 3 commits into
mainfrom
bump-laya/b9fe351c-7b928d82

Conversation

@bong-water-water-bong

Copy link
Copy Markdown
Collaborator

third_party/laya: 7a77439b0923 -> b9fe351ce81e (NandhaKishorM/laya main). config/laya.json revision: 7b928d828b7b -> 7b928d828b7b (convaiinnovations/laya).

Before merging: scripts/fetch-laya.sh

(verifies every file) and the Laya tests.

@bong-water-water-bong
bong-water-water-bong merged commit e35d480 into main Oct 28, 2026
7 of 8 checks passed
@bong-water-water-bong
bong-water-water-bong deleted the bump-laya/b9fe351c-7b928d82 branch October 28, 2026 03:49
@bong-water-water-bong

Copy link
Copy Markdown
Collaborator Author

Correction to my merge note: grep -rn third_party/laya matches four places, not just the workflow — .gitmodules:40, NOTICE:127, docs/laya.md:32 and .github/workflows/bump-laya.yml (plus the workflow's own git ls-tree/update-index lines). None of them compiles or imports the tree (the router builds engine-local laya/, the fast scorer builds third_party/ggmlc), so the risk sentence holds; the wording was too narrow.

One drift worth a follow-up, found while checking: docs/laya.md:32 still records the source as 6d942c92 in a table whose owner column is bump-laya.yml, and that was already stale before this merge (the gitlink was 7a77439b). The bumper moves third_party/laya and config/laya.json but not that table, so the doc has been three revisions behind since it was written. Cheap fix for whoever touches the bumper next: have bump-laya.yml rewrite that row (it already knows both values).

bong-water-water-bong added a commit that referenced this pull request Oct 29, 2026
…ump-laya.yml keeps them

docs/laya.md's "The pin" table names bump-laya.yml as the thing that keeps both rows current,
but bump-laya.yml only moves third_party/laya and config/laya.json -- it never writes the doc.
So both rows had rotted:

  - Source: said `6d942c92`; the gitlink has been `b9fe351c` since #611 and is `fe9a6c35` here.
  - Checkpoints: said revision `55cf4c4e`; config/laya.json has been `7b928d82` since #317
    (37cf390). docs/laya.md was last edited in #488.

Set both to what this branch actually pins, so the doc and the gitlinks agree:

  git ls-tree HEAD third_party/laya -> fe9a6c35023a7d94102e7feaaeaae8857d9d1b07
  config/laya.json revision         -> 7b928d828b7b0e022f929d9bd2e44165aa270148

The durable fix is for bump-laya.yml to rewrite these two rows as it bumps, so they cannot drift
again. I cannot make that change here: pushing a file under .github/workflows needs a token with
the `workflow` scope and this one has only gist/read:org/repo --

  ! [remote rejected] tmp/scope-probe -> tmp/scope-probe (refusing to allow an OAuth App to
    create or update workflow `.github/workflows/bump-laya.yml` without `workflow` scope)
bong-water-water-bong added a commit that referenced this pull request Oct 29, 2026
* Bump Laya: source fe9a6c35023a, model 7b928d828b7b

* Laya: record this bump's pins in docs/laya.md, where the table says bump-laya.yml keeps them

docs/laya.md's "The pin" table names bump-laya.yml as the thing that keeps both rows current,
but bump-laya.yml only moves third_party/laya and config/laya.json -- it never writes the doc.
So both rows had rotted:

  - Source: said `6d942c92`; the gitlink has been `b9fe351c` since #611 and is `fe9a6c35` here.
  - Checkpoints: said revision `55cf4c4e`; config/laya.json has been `7b928d82` since #317
    (37cf390). docs/laya.md was last edited in #488.

Set both to what this branch actually pins, so the doc and the gitlinks agree:

  git ls-tree HEAD third_party/laya -> fe9a6c35023a7d94102e7feaaeaae8857d9d1b07
  config/laya.json revision         -> 7b928d828b7b0e022f929d9bd2e44165aa270148

The durable fix is for bump-laya.yml to rewrite these two rows as it bumps, so they cannot drift
again. I cannot make that change here: pushing a file under .github/workflows needs a token with
the `workflow` scope and this one has only gist/read:org/repo --

  ! [remote rejected] tmp/scope-probe -> tmp/scope-probe (refusing to allow an OAuth App to
    create or update workflow `.github/workflows/bump-laya.yml` without `workflow` scope)

---------

Co-authored-by: laya-bump <laya-bump@users.noreply.github.com>
Co-authored-by: bong-water-water-bong <bong-water-water-bong@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant