Merge pull request #276 from com-pas/sync

Sync from OpenSCD

Author: pascalwilbrink

.github/workflows/stale-issues.yml

@@ -0,0 +1,44 @@
+# This workflow labels stale issues.
+#
+# For more information, see:
+# https://github.com/actions/stale
+name: Mark stale issues
+
+on:
+  workflow_dispatch:
+  schedule:
+    - cron: '0 19 * * *'
+
+jobs:
+  stale:
+    runs-on: ubuntu-latest
+    permissions:
+      issues: write
+
+    steps:
+    - uses: actions/stale@v5
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+
+        days-before-stale: 60
+        days-before-close: -1
+        days-before-pr-stale: -1
+        days-before-pr-close: -1
+
+        stale-issue-label: 'stale'
+        stale-issue-message: |
+          Hello there,
+
+          Thank you for opening this issue! We appreciate your interest in our project.
+          However, it seems that this issue hasn't had any activity for a while. To ensure that our issue tracker remains organized and efficient, we occasionally review and address stale issues.
+
+          If you believe this issue is still relevant and requires attention, please provide any additional context, updates, or details that might help us understand the problem better.
+          Feel free to continue the conversation here.
+
+          If the issue is no longer relevant, you can simply close it. If you're uncertain, you can always reopen it later.
+
+          Remember, our project thrives on community contributions, and your input matters. We're here to collaborate and improve.
+          Thank you for being part of this journey!
+
+
+

docs/decisions/0001-record-architecture-decisions.md

@@ -0,0 +1,28 @@
+# 1. Record architecture decisions
+
+Date: 2023-07
+
+## Status
+
+Accepted
+
+## Context
+
+We need to record the architectural decisions made on this project.
+
+## Decision
+
+We will follow the decisions recorded in the central organizational 
+repository ([.github](https://github.com/openscd/.github)), 
+and record new repo-specific decisions in this repository.
+
+
+We write ADRs in the `docs/decisions` folder instead of a standard `doc/adr`:
+- `docs` instead of `doc` because `doc` is used for the generated documentation.
+- `decisions` instead of `adrs` because it is more explicit and a followed practice:
+  [↗ Markdown Any Decision Records - Applying MADR to your project ](https://adr.github.io/madr/#applying-madr-to-your-project)
+
+## Consequences
+
+- It will be harder to track which decisions have to be taken into consideration
+- Local decisions will be easier to find

src/Hosting.ts

@@ -15,7 +15,8 @@ import { Drawer } from '@material/mwc-drawer';
 import { ActionDetail, List } from '@material/mwc-list';
 import { ListItem } from '@material/mwc-list/mwc-list-item';
 
-import { Mixin, newPendingStateEvent, UserInfoEvent } from './foundation.js';
+import { Mixin, newPendingStateEvent } from './foundation.js';
+import { UserInfoEvent } from './compas/foundation.js';
 import { LoggingElement } from './Logging.js';
 import { PluggingElement, Plugin, pluginIcons } from './Plugging.js';
 import { SettingElement } from './Setting.js';

src/compas/CompasUserInfo.ts

@@ -1,4 +1,4 @@
-import { newUserInfoEvent } from '../foundation.js';
+import { newUserInfoEvent } from './foundation.js';
 
 import { CompasUserInfoService } from '../compas-services/CompasUserInfoService.js';
 import { createLogEvent } from '../compas-services/foundation.js';

src/compas/foundation.ts

@@ -106,3 +106,25 @@ export function compareVersions(
   }
   return result;
 }
+
+/** Represents user information from a backend. */
+export interface UserInfoDetail {
+  name: string;
+}
+export type UserInfoEvent = CustomEvent<UserInfoDetail>;
+export function newUserInfoEvent(
+  name: string,
+  eventInitDict?: CustomEventInit<Partial<UserInfoDetail>>
+): UserInfoEvent {
+  return new CustomEvent<UserInfoDetail>('userinfo', {
+    bubbles: true,
+    composed: true,
+    ...eventInitDict,
+    detail: { name, ...eventInitDict?.detail },
+  });
+}
+declare global {
+  interface ElementEventMap {
+    ['userinfo']: UserInfoEvent;
+  }
+}

src/editors/IED.ts

@@ -96,7 +96,11 @@ export default class IedPlugin extends LitElement {
     super.updated(_changedProperties);
 
     // When the document is updated, we reset the selected IED.
-    if (_changedProperties.has('doc') || _changedProperties.has('nsdoc')) {
+    if (
+      _changedProperties.has('doc') ||
+      _changedProperties.has('editCount') ||
+      _changedProperties.has('nsdoc')
+    ) {
       this.selectedIEDs = [];
       this.selectedLNClasses = [];
 

src/editors/communication/subnetwork-editor.ts

@@ -142,9 +142,7 @@ export class SubNetworkEditor extends LitElement {
   private subNetworkSpecs(): string {
     if (!this.type && !this.bitrate) return '';
 
-    return `(${this.type}${
-      this.type && this.bitrate ? ` — ${this.bitrate}` : ``
-    })`;
+    return `(${[this.type, this.bitrate].filter(text => !!text).join(' — ')})`;
   }
 
   private header(): string {

src/editors/protocol104/wizards/selectDo.ts

@@ -13,7 +13,7 @@ import {
   getNameAttribute,
   identity,
   newSubWizardEvent,
-  selector,
+  find,
   Wizard,
   WizardActor,
   WizardInputElement,
@@ -158,7 +158,7 @@ function checkAndGetLastElementFromPath(
   const [tagName, id] = path.pop()!.split(': ');
   if (!expectedTag.includes(tagName)) return null;
 
-  return doc.querySelector(selector(tagName, id));
+  return find(doc, tagName, id);
 }
 
 /**

src/editors/publisher/data-set-editor.ts

@@ -19,7 +19,7 @@ import './data-set-element-editor.js';
 import '../../filtered-list.js';
 import { FilteredList } from '../../filtered-list.js';
 
-import { compareNames, identity, selector } from '../../foundation.js';
+import { compareNames, identity, find } from '../../foundation.js';
 import { styles, updateElementReference } from './foundation.js';
 
 @customElement('data-set-editor')
@@ -50,7 +50,7 @@ export class DataSetEditor extends LitElement {
 
   private selectDataSet(evt: Event): void {
     const id = ((evt.target as FilteredList).selected as ListItem).value;
-    const dataSet = this.doc.querySelector(selector('DataSet', id));
+    const dataSet = find(this.doc, 'DataSet', id);
 
     if (dataSet) {
       this.selectedDataSet = dataSet;

src/editors/publisher/foundation.ts

@@ -1,6 +1,6 @@
 import { css } from 'lit-element';
 
-import { identity, selector } from '../../foundation.js';
+import { identity, find } from '../../foundation.js';
 
 export function updateElementReference(
   newDoc: XMLDocument,
@@ -9,7 +9,7 @@ export function updateElementReference(
   if (!oldElement || !oldElement.closest('SCL')) return null;
 
   const id = identity(oldElement);
-  const newElement = newDoc.querySelector(selector(oldElement.tagName, id));
+  const newElement = find(newDoc, oldElement.tagName, id);
 
   return newElement;
 }