Conversation
github-issue-44 Add numbering-new-symbol invariant and NumberingSequence NewSymbol insert tasks. Patch §V.40 seed field set: NewSymbol is seed; LastNbr stays runtime. Record §B.38.
5 tasks
…ion bump 1.10.0→1.11.0 V21: field-shape change in packaged bootstrap_project.xml bumps Bootstrap 1.10.0 → 1.11.0; sweep src/ + tests/ Bootstrap/<ver> pins. V58/B38: map NewSymbol to Header (CS201010 New Number Symbol). UserNumbering is the OR alternative, not a required pair. Do not map NewSymbol to Sequence (T159-class 422).
V34: catalog include lists NewSymbol now that package seed has it. V40: LastNbr stays runtime (NUMBERING_RUNTIME_FIELDS); NewSymbol is seed. V56/V58: GET omits NewSymbol → diff-ignore, not not-returned; apply still PUTs; extract emits when GET returns (not LastNbr strip).
V13: gmake check. V40: LastNbr extract/diff/apply strip stands. V56/V58: apply PUTs NewSymbol; GET-omit is not not-returned drift; extract emits NewSymbol when GET returns. Contract maps Header.
V4/V58/B38: throwaway T254NS insert with NewSymbol: <NEW> and Descr (QORD-class) does not 422. Re-apply of existing BATCH and package sequences succeeds (no NewSymbol mask 422 on update).
V12/V19/V58: demo-seed bounds vs runtime names NewSymbol as seed (insert-required) and LastNbr as runtime. CHANGELOG Unreleased records gh #44 Bootstrap 1.11.0 Header map.
Mapped GET after Bootstrap 1.11.0 returns NewSymbol. Diff now compares it so value drift flags. E2e asserts the GET shape. Drop narrative comments on the LastNbr set and extract strip.
kborovik
marked this pull request as ready for review
September 9, 2026 20:02
5 tasks
Select only `AcctCD` in the Company list query for open-periods synthesis to avoid 500 errors caused by unprojected BQL-delegate fields (`DecPlQty`, `WeightUOM`, `VolumeUOM`).
Live e2e is provision, scenario, and extract. Per-bug probes fold onto those tenants. A new e2e file is only for a different tenant shape.
Share bracket_tenant and joined_output in conftest. Fold DecPlQty, roles SQL, NewSymbol insert, and 26-char InventoryID into provision. Fold KitAssembly alloc into scenario after buy/build. Drop five per-bug files. Offline test locks the three-file set.
This CLI repo now owns the ACUCLI test seed under repo-root config/ and scenario/. Product GitOps stays in the sibling data repo. Soak runs from this checkout with --tenant ACUCLI. Packaged templates still feed gmake e2e. Dropped V2 clause "both live in data repos, not here".
…ws} + scenario/ to repo root (exclude qms/, live .env, state/, customization, Makefile, schemas) Copy the CLI test seed from sibling GitOps into this checkout. Prune qms/, live .env, state/, customization, Makefile, and schemas. §V.2
…CLI; never CNBN; never sibling cd for CLI soak; e2e still packaged templates Soak the CLI test tenant from this checkout. Do not cd the sibling GitOps repo. gmake e2e still uses packaged config init templates. §V.2 §V.12
… still 3 pipeline files Assert CLI test seed dirs exist, config/qms is absent, and live e2e stays three pipeline files. §V.2 §V.13
…-omit comments Missing live config now tells operators to decrypt .env.gpg here. Diff ignore-field comments keep V56 on path-qualified GET-omit only. Contract comment matches mapped GET returning NewSymbol. §V.2 §V.56
…d drift Pruned GET-omit write-only clause. That was the unmapped 1.10.0 observation. After Header map, GET returning NewSymbol is the mapping-miss detector. LastNbr stays the only numbering runtime strip.
…SoT; .env-only live instance; drop retired sibling/host literals Drop the docs/ tree as a live source of truth. Live instance URL comes from .env only. Drop retired sibling and host names: acumatica-infra, acumatica-blog, acumatica-devops, acu-dev1.vm.internal.
Drop the docs/ tree. Live verify stays on .env and dumped schema, not a docs/ source of truth. Sweep remaining docs/*.md cites from src/ and tests. Human-facing README and CHANGELOG Unreleased links wait for T262. §V.12 §V.49
Replace the README example ACU_BASE_URL with the erp.example.com placeholder. Live instance stays in .env only. Drop the retired acumatica-infra sibling name from the app-pool docstring. §V.12 §V.27
check-md-prose and the V49 extras recipe scan README plus packaged template README only. Drop leftover docs/*.md links from README. Unreleased records the docs/ tree removal. Root help already points at the package README. §V.49 §V.12 §V.19
CLI test-seed numbering sequences now carry NewSymbol: <NEW>, matching the packaged template so a cold insert does not 422. The bootstrap contract header no longer claims data-repo project.xml is a fallback. Packaged bootstrap_project.xml is the only contract. §V.58 §V.21
Archive closed tasks T133 through T212 into SPEC.archive.md. Keep the newest 50 live (T213 through T262). Extract heavy invariant audit recipes into .spec/check-extras.md.
Ensure e2e tests resolve the repository's venv binary and `.env` file instead of ambient process environment variables. Allow clean extract skips for empty role membership GETs in roundtrip extraction tests.
kborovik
deleted the
44-bootstrap-numberingsequence-insert-422-newsymbol-not-mapped
branch
September 10, 2026 19:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fold GitHub issue #44 into SPEC.md. NumberingSequence insert requires Header NewSymbol.
Changes
Related: #44
Closes #44