Skip to content

Phase 10 Addendum AD01 / Task AD2: Integrate Advanced Upstream and Resolve Coexistence #202

Closed

Description

@Sam-Bolling

Planning authority:


  • Objective: Merge the amended AD1-locked upstream commit into workbench phase-10 and produce one coherent result that preserves the complete approved upstream range and the already-approved CSAPI contribution.
  • Acceptance criteria advanced: P10-AD01-A1, P10-AD01-A2, P10-AD01-A3, P10-AD01-B1, P10-AD01-B2, P10-AD01-B3, P10-AD01-B4, P10-AD01-B5.
  • Dependencies: AD1 completed without a stop condition.
  • Complexity/risk: High; foundational workbench merge and three authored conflict resolutions.
  • Locked decisions: Normal --no-ff merge; exact AD1 SHA; no whole-file ours/theirs; add only hasConnectedSystems?: boolean to upstream's capability interface; preserve dynamic imports and subpath boundary.
  • Expected work:
    • verify the clean workbench state and the AD1 evidence immediately before mutation;
    • merge exactly a623911201218bc1e814a9f83c64f3a027031990;
    • resolve app/api.data.js by combining upstream property/static-method/constructor handling with Phase 10 per-entry-point import mapping and combined TypeDoc modules;
    • resolve src/ogc-api/endpoint.ts by combining upstream capability/options types and documentation with the CSAPI getters, facade, errors, and dynamic imports;
    • resolve src/ogc-api/info.ts using upstream's OgcApiCollectionCapabilities while retaining CSAPI conformance and collection-link detection;
    • add hasConnectedSystems?: boolean to OgcApiCollectionCapabilities in src/ogc-api/model.ts;
    • inspect the automatic src/index.ts result for all reviewed upstream and Phase 10 exports and the absence of CSAPI bulk root exports;
    • run focused type, endpoint/info test, documentation-generation/build, source-assertion, formatting, and diff checks;
    • commit the merge and push phase-10 normally.
  • Affected areas: Repository merge result, with authored resolution/adaptation limited to app/api.data.js, src/ogc-api/endpoint.ts, src/ogc-api/info.ts, and src/ogc-api/model.ts; mandatory inspection of src/index.ts.
  • Explicit exclusions: New API behavior, facade redesign, static CSAPI runtime imports, new exports beyond the one capability field, unrelated changes to the approved upstream range, broad formatting, live testing, delivery changes.
  • Automated acceptance gate:
    • ancestry check for the locked upstream SHA exits 0;
    • no unmerged paths remain and actual conflicts equal the AD1 forecast;
    • Node.js 24 typecheck exits 0;
    • existing info.spec.ts and endpoint.spec.ts tests pass;
    • npm run docs:api and npm run docs:build exit 0;
    • source assertions confirm the capability field, dynamic imports, reviewed root exports, and root/subpath boundary;
    • explicit formatting and git diff --check pass;
    • workbench Actions succeeds for the pushed merge; and
    • the final authored resolution population contains no unexplained path.
  • Deliverable: One pushed normal workbench merge commit containing the reviewed coexistence resolutions.
  • Closing evidence: Merge SHA, upstream SHA, resolved/companion/inspected paths, focused gate totals and results, Actions URL, deviations or none.

Activity

  1. Sam-Bolling commented on Aug 28, 2026

    @Sam-Bolling
    Author

    Entry stop condition reached before mutation.

    At issue entry, fetch advanced canonical upstream/main from the AD1-locked 00a8931c679fb9c5cd7c5e0f0f04a4b87a307b2d to a623911201218bc1e814a9f83c64f3a027031990. This differs from #201's locked canonical-upstream disposition, so the approved Guide/Roadmap prohibit beginning the merge without project-owner reassessment.

    Read-only assessment:

    • 4 new commits after the locked SHA;
    • 5 changed paths: app/index.md, src/index.ts, src/ogc-api/model.ts, src/shared/url-utils.ts, and src/stac/model.ts;
    • current-tip merge forecast still reports exactly the same three content conflicts (app/api.data.js, src/ogc-api/endpoint.ts, src/ogc-api/info.ts);
    • src/index.ts still merges automatically;
    • no new textual conflict was introduced;
    • workbench remains clean at 885056b9f795eda1536814a1afb887eb2c2934c0, equal to origin/phase-10;
    • no merge, source edit, package install, delivery mutation, commit, push, or PR change occurred.

    Recommended disposition: explicitly advance the AD01 baseline to a623911201218bc1e814a9f83c64f3a027031990, reopen/update AD1 evidence rather than create another issue or planning trio, then resume this issue after the amended lock is accepted.

  2. changed the title [-]Phase 10 Addendum AD01 / Task AD2: Integrate PR #171 and Resolve Coexistence[/-] [+]Phase 10 Addendum AD01 / Task AD2: Integrate Advanced Upstream and Resolve Coexistence[/+] on Aug 28, 2026
  3. Sam-Bolling commented on Aug 28, 2026

    @Sam-Bolling
    Author

    AD1 amendment is complete at 28ef8846c87c084a51be13c6e733997855765c06; baseline a623911201218bc1e814a9f83c64f3a027031990 is locked, QA passed, and this issue is resuming under its amended description.

  4. Sam-Bolling commented on Aug 28, 2026

    @Sam-Bolling
    Author

    Completed and accepted.

    Merge commit: 4acedf4cb5a3e38fd3cb45195cbba0ec5fe40406

    Parents:

    • workbench: 28ef8846c87c084a51be13c6e733997855765c06;
    • locked upstream: a623911201218bc1e814a9f83c64f3a027031990.

    Resolution population:

    • app/api.data.js: retained upstream property/static-method/no-constructor handling and Phase 10 root/CSAPI module import-path mapping;
    • src/ogc-api/endpoint.ts: retained upstream capability/options types and documentation plus CSAPI getters, facade, errors, and both dynamic imports;
    • src/ogc-api/info.ts: adopted upstream OgcApiCollectionCapabilities while retaining CSAPI conformance and collection-link detection;
    • src/ogc-api/model.ts: added only the authorized hasConnectedSystems?: boolean companion field;
    • src/index.ts: automatic result inspected; all reviewed upstream exports and DateTimeParameter remain, with no bulk CSAPI root export.

    Gates:

    • actual conflicts: exact approved three; no unmerged paths;
    • locked-upstream ancestry: PASS;
    • first-parent changed-path inventory: exact 32-row match to AD1;
    • upstream-overlay population: exact five reviewed paths (app/api.data.js, src/index.ts, src/ogc-api/endpoint.ts, src/ogc-api/info.ts, src/ogc-api/model.ts);
    • Node.js 24.20.0 npm ci: PASS; 836 packages; inherited audit/install-script notices recorded, no remediation;
    • Node.js 24 typecheck: PASS;
    • local focused suites: 96/97 passed; the sole miss was the pre-existing invalid-JSON fixture receiving CRLF from Windows core.autocrlf=true instead of its LF Git blob;
    • npm run docs:api: PASS; 0 errors / 10 inherited warnings;
    • npm run docs:build: PASS;
    • representative CSAPI TypeDoc elements and /csapi import path: PASS;
    • package root and ./csapi exports: PASS;
    • explicit formatting and git diff --check: PASS;
    • dynamic-import/static-import boundary assertions: PASS;
    • Linux Quality Assurance run 33175454920: PASS for Node 24 install, formatting, typecheck, lint, browser tests, Node tests, docs build, and diff check. This provides the authoritative LF-platform pass for the focused test affected locally by checkout conversion.

    Final state: phase-10 is clean, synchronized with origin/phase-10, and contains the amended locked upstream commit.

    Deviation from locked decision: none. Task AD3 (#203) is dependency-ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions