Skip to content

audit: apollo3 / framework-arduinoambiqapollo3 — does platform_packages override work? (parent #664) #665

Closed

Description

@zackees

Parent: #664 (full context, scope, motivation).

What to check

  1. Open crates/fbuild-packages/src/library/apollo3_core.rs — confirm the framework URL / commit / sha256 are pinned as const &str and that the only public constructor takes no override.
  2. Open the orchestrator at crates/fbuild-build/src/apollo3/orchestrator.rs — does it consult config.get_env_config(env_name)? for a platform_packages entry naming this framework, and pass the URL+ref through to a with_override constructor? Or does it only ever call the default constructor?
  3. Reply on this issue with:
    • bug exists — if the orchestrator never reads platform_packages, plus file:line for both the const URL pin and the call site.
    • fixed / no bug — if an override path exists, plus file:line showing the lookup + override constructor.

Reference

#663 is the original report of this pattern (nxplpc). #664 lists every framework package being audited. Use the same evidence shape: file:line for the const URL pin and for the orchestrator call site.

Activity

  1. zackees commented on Jun 20, 2026

    @zackees
    MemberAuthor

    VERDICT: bug exists

    EVIDENCE:

    • apollo3_core.rs:15-16 — const APOLLO3_CORE_URL: &str = "https://github.com/sparkfun/Arduino_Apollo3/releases/download/v2.2.1/Arduino_Apollo3.tar.gz"
    • apollo3_core.rs:25-38 — only constructor pub fn new(project_dir: &Path) hard-codes APOLLO3_CORE_URL
    • apollo3_core.rs:41-55 — test constructor with_cache_root() also uses hard-coded URL
    • crates/fbuild-build/src/apollo3/orchestrator.rs:56 — Apollo3Cores::new(...) called with only project_dir, no platform_packages override
    • crates/fbuild-packages/src/lib.rs:205-227 — PackageBase::new() accepts no URL override

    NOTES: Apollo3 ignores env_config.get("platform_packages"); identical to nxplpc bug pattern from #663.

  2. added a commit that references this issue on Jun 30, 2026
    b6246ff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions