Skip to content

Add the Altinn landscape generator and update the Apps drawings - #3131

Merged
TheTechArch merged 1 commit into
masterfrom
feature/altinn-landscape-generator
Sep 29, 2026
Merged

TheTechArch merged 1 commit into
masterfrom
feature/altinn-landscape-generator

Conversation

@TheTechArch

@TheTechArch TheTechArch commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Two follow-ups to the Altinn landscape page (#3127, #3129, #3130).

The generator is now in the repository

scripts/altinn-landscape/generate.js builds all the drawings on the page. It reads the file list of origin/main in each Altinn repository with git ls-tree, so the local branch does not matter and nothing in the repositories changes. It has no npm dependencies.

node scripts/altinn-landscape/generate.js --repos C:/repos --fetch

If a link points to a path that no longer exists on main, the script lists it and exits with code 1. scripts/altinn-landscape/README.md explains how to run the script, which repositories it needs, how it is structured and how to fix missing paths. The landscape page links to the README and notes that manual edits in draw.io are overwritten on the next run.

Apps now points to the app backend in altinn-studio

The app backend has moved to altinn-studio/src/App/backend, and app-lib-dotnet is no longer updated (last commit 2026-09-21). The Apps drawings now link there and reflect the current code:

  • the workflow engine integration (WorkflowEngineService, WorkflowEngineCallbackController) is added to the process engine column
  • ProcessEventDispatcher, EformidlingStartup and FiksArkivHost are replaced by the workflow engine, EFormidlingStatusReader and FiksArkivSubscriber
  • instance locking is removed (replaced by the aggregate instance-mutation endpoint, #19919 in altinn-studio)
  • the Altinn Events receive stack is removed (#19859); apps still publish events
  • PagesController and TextClient are removed as obsolete APIs

The app frontend still links to app-frontend-react, which is still active.

The other drawings change only because frames below Apps moved down slightly. Their content is the same, except that the PEP "Used by" link now points to altinn-studio.

Checked with a local hugo build: no errors.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Clarified that the landscape diagrams are generated from repository source code and that manual edits are overwritten when they are regenerated.
    • Added guidance on regenerating diagrams, command options, output files and validation errors.
    • Specified that app-lib is in the Altinn Studio repository.

Adds scripts/altinn-landscape/generate.js, which builds all landscape drawings
from origin/main of the Altinn repositories with git ls-tree, and a README on
how to run it and how to fix paths it reports as missing. The script exits
with an error when a link points to a path that no longer exists on main.

The app backend has moved to altinn-studio/src/App/backend, so the Apps
drawings now link there and reflect its current code: the workflow engine
integration replaces the process event dispatcher and the Fiks Arkiv and
eFormidling hosted loops, instance locking and the Altinn Events receive stack
are gone, and obsolete APIs such as PagesController and TextClient are dropped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: Altinn/altinn-studio-docs/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 2e216dd2-8d1c-461f-8cf2-774c91d744d9

📥 Commits

Reviewing files that changed from the base of the PR and between 4abde96 and f9780c3.

⛔ Files ignored due to path filters (8)
  • content/technology/architecture/altinn-landscape/altinn_apps_detailed.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_authorization_detailed.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_correspondence_broker_detailed.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_events_notifications_detailed.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_overview.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_profile_detailed.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_studio_detailed.drawio.svg is excluded by !**/*.svg
  • content/technology/architecture/altinn-landscape/altinn_super_detailed.drawio.svg is excluded by !**/*.svg
📒 Files selected for processing (4)
  • content/technology/architecture/altinn-landscape/_index.en.md
  • content/technology/architecture/altinn-landscape/_index.nb.md
  • scripts/altinn-landscape/README.md
  • scripts/altinn-landscape/generate.js
 ___________________________________________________________________________________________________________________________________
< Finish what you start. Where possible, the routine or object that allocates a resource should be responsible for deallocating it. >
 -----------------------------------------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@TheTechArch
TheTechArch merged commit ea02163 into master Sep 29, 2026
5 of 6 checks passed
@TheTechArch
TheTechArch deleted the feature/altinn-landscape-generator branch September 29, 2026 14:21
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